-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[undecorate] partial fix for #3460: @action => override #3478
Conversation
🦋 Changeset detectedLatest commit: 15879ff The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
Great, now it fails on facebook/jscodeshift#424 |
@@ -217,22 +217,22 @@ Note that it is possible to pass `{ proxy: false }` as an option to `observable` | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably done by precommit pretty hook
@@ -85,6 +85,7 @@ It's a bit opinionated and can lead to a lot of false positives depending on you | |||
### mobx/no-anonymous-observer (deprecated) | |||
|
|||
_Deprecated in favor of [react/display-name](https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/display-name.md) + [componentWrapperFunctions](https://github.com/jsx-eslint/eslint-plugin-react). Example of **.eslintrc**:_ | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably done by precommit pretty hook
UPDATE: Fixed by upgrading jscodeshift.override
keyword isn't preserved forkeepDecorators: false
. Not sure if it's due to #1151 or if there an issue on our side.That's why test are failing, but it's possible to adapt them.
Does not support
decorate
May leave unused
action
import.