fix: replace resolveString with verifyString

[monbrey]

Replaces a reference to the now removed Util.resolveString with Util.verifyString
Introduced by discordjs/discord.js#4880

[ghost]

I just tested this PR and looks like this.options is giving the error TypeError: Cannot read property 'content' of undefined, without the this.options is working fine

[ghost]

Take out this.options

[ghost]

Shouldn't StringResolvable be Replaced with string in the Typings?

[kyranet]

StringResolvable is being used for message sending and editing, both of which are changing in discordjs/discord.js#5758

Once the PR has landed, this can be updated and fix two breaking changes together.

[monbrey]

I don't believe this PR should address both changes, no. I will update the typing though.

[kyranet]

Oh, yeah mb, I thought this was a PR for updating Commando for v13.

[ghost]

Oh, yeah mb, I thought this was a PR for updating Commando for v13.

There are a lot of things missing to support v13, This is getting kinda off-topic lets discuss on library-discussion if You want.

[arifali123]

can this be merged so we can use commando with the master version of discord.js

[ghost]

This PR just Broke lmao, now You get verifyString is not a function

[kyranet]

It's still in there: https://github.com/discordjs/discord.js/blob/master/src/util/Util.js#L403

Are you sure you didn't change back to 12.5.3?

[ghost]

Make Separate imports to escapeMarkdown, splitMessage, verifyString
const { // Imports } = require('discord.js').Util;

[ghost]

It's still in there: https://github.com/discordjs/discord.js/blob/master/src/util/Util.js#L403

Are you sure you didn't change back to 12.5.3?

Yea I didn't changed back to 12, He Just forgot to add .Util after requiring d.js