Conversation
…c-await-events-and-no-await-sync-events
MichaelDeBoey
left a comment
There was a problem hiding this comment.
We should release this in a major release btw as this is breaking behavior for this specific rule and people need to adjust their config if they want the current released functionality (for instance when only enabling this specific rule)
Co-authored-by: Michaël De Boey <info@michaeldeboey.be> Signed-off-by: Mario Beltrán <belco90@gmail.com>
Not really. In our semantic version policy, we specify that a bug fix in a rule that results in eslint-plugin-testing-library reporting fewer errors will be a patch release. The rule was reporting both fire-event and user-event. The change done here is to report only fire-event by default. The changes in the presets were needed to reflect the new default change correctly. However, the actual config of the rule and the way it reports the errors are the same. I get the point of your comment, tho. A change in the number of errors reported could potentially be a breaking change. That's why we defined the semver policy, inspired by ESLint semver policity itself, so users know what to expect. |
|
🎉 This PR is included in version 6.0.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Checks
Changes
user-eventmethods by default, assuming v14 is the standard now.Context
Fixes #801
Fixes #669