-
Notifications
You must be signed in to change notification settings - Fork 40
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
Parity with eslint-plugin-jest
#67
Comments
eslint-plugin-jest
eslint-plugin-jest
eslint-plugin-jest
@mskelton hey, would you mind undoing the reformatting? I was using it to visual scan for changes from the jest dir. I had 2 i needed to add from their recent deployment |
@unlikelyzero The reason I changed it was so we can use GitHub's feature to convert a list item into an issue for tracking. Will make it much easier to see progress as we work through the list creating and closing issues for each of these features. Can we just add the new rules to the end of the list? |
@mskelton in this case, I think we should have a near-synchronous discussion on the the purpose of this ticket. this ticket initially was supposed to be a one-time audit and a conversation about what is applicable to this project. I think all of that work has been undone |
After a year and a half of opening this issue, I'm happy to report I've finished the last of the remaining lint rules! |
We're establishing coding practices for our e2e suite before opening it up to the public. Generally, we're looking to enforce these code practices wherever possible and would likely need most of the rules from the Jest ESLint plugin.
Probably don't need these
The text was updated successfully, but these errors were encountered: