-
Notifications
You must be signed in to change notification settings - Fork 154
feat(prefer-user-event-setup): add new rule #1125
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
base: main
Are you sure you want to change the base?
feat(prefer-user-event-setup): add new rule #1125
Conversation
Enforces using userEvent.setup() pattern instead of direct method calls as recommended in the official user-event documentation.
3626415 to
f89ad18
Compare
❌ 1 Tests Failed:
View the top 1 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
Belco90
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gipcompany thanks for your contribution! Great implementation and test cases, just added some minor comments.
Signed-off-by: Atsushi Ishida <gipcompany@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The rule must be manually exported in lib/rules/index.ts
Checks
Changes
Context
Closes #646