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

eslint-plugin-istanbul #477

Closed
coreyfarrell opened this issue May 14, 2020 · 3 comments
Closed

eslint-plugin-istanbul #477

coreyfarrell opened this issue May 14, 2020 · 3 comments

Comments

@coreyfarrell
Copy link
Contributor

A few days ago I published eslint-plugin-istanbul. I'm not sure if this is something xo would be interested in adding to the next major (either as an error or just a warning). Feel free to close this if you are not interested, just figured it was worth announcing.

@sindresorhus
Copy link
Member

prefer-ignore-reason could maybe be useful. However, the plugin is fairly immature and I don't want to deal with early issues, so let's revisit this later when the plugin is more mature.

@coreyfarrell
Copy link
Contributor Author

This is reasonable, thanks for responding. Up to you if you want to close this issue or leave it open so users can potentially see it and test on their own.

Just to share the current plan I think prefer-ignore-reason and no-ignore-file will never change what they report. prefer-ignore-reason will never be fixable. I'm not sure if no-ignore-file should be fixable or not. If that decision is made it will be made before 1.0.0 and will be listed as a breaking change.

@fregante
Copy link
Member

Plugin dormant. Useful rule covered by #571

@fregante fregante closed this as not planned Won't fix, can't repro, duplicate, stale Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants