feat: migrate all message to messageId

[scagood]

Migrate all message to messageId (for the update to eslint-plugin-eslint-plugin)

(#471)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (bbcfcbf) to head (29130f1).
Report is 11 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #482   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           25        24    -1     
  Lines          649       660   +11     
  Branches       250       246    -4     
=========================================
+ Hits           649       660   +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[brettz9]

Was this one supposed to be as a draft too?

[brettz9]

I like most of the names here (better than the ones I was attempting in my own PR), but just those few mentioned could I think be improved, with the naming being important since they are meant to be accessed even outside the file containing them as in the test files (so the test files can avoid specifying an exact syntax which might be contradicted by a change of the message in the rule file).

[brettz9]

LGTM!

[brettz9]

It looks per https://github.com/eslint-community/governance/pull/1/files that collaborators (and not only team members) can do merges (presumably on other-than-their-own PRs), so I'll go ahead and merge.

[brettz9]

I changed from feat to refactor, as it seems there are no user-facing changes.

[github-actions]

🎉 This PR is included in version 6.5.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀