Skip to content
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

Fix no-new-mixins #1099

Merged
merged 1 commit into from
Dec 14, 2019
Merged

Fix no-new-mixins #1099

merged 1 commit into from
Dec 14, 2019

Conversation

chancancode
Copy link
Member

@chancancode chancancode commented Dec 14, 2019

A new version of the eslint-plugin-ember caught a few more cases in this file. Since it's intentional here, we should just disable the rule for the whole file.

@chancancode chancancode force-pushed the fix-lint branch 3 times, most recently from 0230f6e to c6c53cf Compare December 14, 2019 19:27
A new version of the `eslint-plugin-ember` caught a few more cases
in this file. Since it's intentional here, we should just disable
the rule for the whole file.
@RobbieTheWagner RobbieTheWagner merged commit 6ab667e into master Dec 14, 2019
@RobbieTheWagner RobbieTheWagner deleted the fix-lint branch December 14, 2019 20:03
nummi pushed a commit to nummi/ember-inspector that referenced this pull request Apr 1, 2020
A new version of the `eslint-plugin-ember` caught a few more cases
in this file. Since it's intentional here, we should just disable
the rule for the whole file.
nummi pushed a commit to nummi/ember-inspector that referenced this pull request Apr 5, 2020
A new version of the `eslint-plugin-ember` caught a few more cases
in this file. Since it's intentional here, we should just disable
the rule for the whole file.
patricklx pushed a commit to patricklx/ember-inspector that referenced this pull request Sep 19, 2022
A new version of the `eslint-plugin-ember` caught a few more cases
in this file. Since it's intentional here, we should just disable
the rule for the whole file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants