fix: incorrectly-used violation message placeholders

[bmish]

Fixing some lint violations detected by my WIP PR to update eslint-plugin/no-missing-placeholders and eslint-plugin/no-unused-placeholders to handle placeholders in messageIds:

• Update no-missing-placeholders and no-unused-placeholders to handle messageIds eslint-community/eslint-plugin-eslint-plugin#252

lib/rules/no-hooks-from-ancestor-modules.js
  31:43  error  The placeholder {{hookName}} does not exist      eslint-plugin/no-missing-placeholders
  31:43  error  The placeholder {{invokedMethodName}} is unused  eslint-plugin/no-unused-placeholders

lib/rules/no-only.js
  27:26  error  The placeholder {{callee}} is unused  eslint-plugin/no-unused-placeholders

lib/rules/no-skip.js
  28:26  error  The placeholder {{callee}} is unused  eslint-plugin/no-unused-placeholders

[coveralls]

Coverage remained the same at 100.0% when pulling 546345f on bmish:fix-message-ids into a785e87 on platinumazure:master.

[bmish]

@raycohen the result of this mistake was it was always printing:

  17:5  error  Do not call hooks.{{hookName}} from an ancestor module  qunit/no-hooks-from-ancestor-modules

[platinumazure]

@bmish Thanks for this! Any reason not to merge this PR? (It's still in Draft state.)

[bmish]

Ready to merge.

[platinumazure]

Thanks again!