mobx icon indicating copy to clipboard operation
mobx copied to clipboard

fix(eslint-plugin-mobx)!: invert exhausive-make-observable autofix

Open kade-robertson opened this issue 1 year ago • 2 comments

To avoid changing behaviour of existing logic, the autofix for this rule should specify a false annotation to avoid changing code behaviour.

Code change checklist

  • [x] Added/updated unit tests -- There were none before and this change is pretty minor.
  • [x] Updated /docs. For new functionality, at least API.md should be updated (Just updated the README.md in this case)
  • [x] Verified that there is no significant performance drop (yarn mobx test:performance) -- I don't believe this is relevant for this change

Closes #3876

kade-robertson avatar May 17 '24 18:05 kade-robertson

🦋 Changeset detected

Latest commit: 7341f711ac5fb63415272ec234f9ebe16aa9161c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
eslint-plugin-mobx Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar May 17 '24 18:05 changeset-bot[bot]

Hi, please see https://github.com/mobxjs/mobx/issues/3876#issuecomment-2119001806 Feel free to improve wording of error message.

urugator avatar May 18 '24 21:05 urugator