fix(escapeMarkdown): fix double escaping

[RedGuy12]

Please describe the changes this PR makes and why it should be merged:

Fixes escapeMarkdown double-escaping everything. This happened because escapeEscapes was called after most things were already escaped.

  if (inlineCode) text = escapeInlineCode(text);
  if (codeBlock) text = escapeCodeBlock(text);
  if (italic) text = escapeItalic(text);
  if (bold) text = escapeBold(text);
// text === "foo_bar"
  if (underline) text = escapeUnderline(text);
// text === "foo\\_bar"
  if (strikethrough) text = escapeStrikethrough(text);
  if (spoiler) text = escapeSpoiler(text);
  if (escape) text = escapeEscape(text);
// text === "foo\\\\_bar"
  if (heading) text = escapeHeading(text);
  if (bulletedList) text = escapeBulletedList(text);
  if (numberedList) text = escapeNumberedList(text);
  if (maskedLink) text = escapeMaskedLink(text);

Regressed by #8701
Status and versioning classification:

• Code changes have been tested against the Discord API, or there are no code changes

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments

Name	Status	Preview	Updated
discord-js	⬜️ Ignored (Inspect)		Nov 19, 2022 at 9:02PM (UTC)
discord-js-guide	⬜️ Ignored (Inspect)		Nov 19, 2022 at 9:02PM (UTC)

[KevinNovak]

Thanks for working on this! Looking forward to the merge!

[JPBM135]

Shouldn't tests be changed? Or add tests for this?