-
Notifications
You must be signed in to change notification settings - Fork 15
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
Remove auto-fix #27
Comments
Hey @levaleks, I think in general test IDs don't need to be semantic, they just need to be uniquely addressable so automated tests can refer to them. Random IDs were a big help for our QA team, but I recognize that was just one way of working. Thanks for the feedback, and I for one would be interested to understand more about the issues this is causing you. I would be interested in adding a |
Hi! As I told you before, thank you for a great plugin.
There is one thing that I haven't noticed earlier (it's only mentioned in the changelog) but become a big deal now. AFAIK there is no way to disable the auto-fix option per plugin on the
.eslintrc
level by design. Many projects have lint-staged witheslint --fix
option, or auto fix enabled on save in IDE, and the last thing you want to see in the codebase is a random unconfigurable testid :(It's a big no-go for adoption :/
cc @bkonuwa @pixelbandito
The text was updated successfully, but these errors were encountered: