feat: add basic config for cypress #117
Conversation
| rules: { | ||
| 'testing-library/no-debug': 'warn', | ||
| }, |
There was a problem hiding this comment.
Notice here that we don't spread recommendedRules
There was a problem hiding this comment.
I guess we should update the comment in Frameworks section within README saying "Note that frameworks configurations enable their specific rules + recommended rules."
There was a problem hiding this comment.
I appreciate you started the cypress preset. However, I think we should implement at least the prefer-explicit-assert adaptation for cypress mentioned here. There is an upcoming migration to TS for this repo, I don't know if you prefer to wait until then.
| rules: { | ||
| 'testing-library/no-debug': 'warn', | ||
| }, |
There was a problem hiding this comment.
I guess we should update the comment in Frameworks section within README saying "Note that frameworks configurations enable their specific rules + recommended rules."
| Ah ok, I missed that thread. No problem, let me close this PR then. There is no rush for this. Thanks :) |
2 participants
I noticed that some of the default rules (like for
async) don't really work for Cypress, as commands are not promises.So I thought it would be good to start a default config for Cypress as well.