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

Extend eslint - jest/valid-expect - to allow 2nd optional argument for message. #6

Closed
alee8 opened this issue Dec 3, 2018 · 2 comments · Fixed by jest-community/eslint-plugin-jest#584 or #32

Comments

@alee8
Copy link

alee8 commented Dec 3, 2018

Feature Request

Description: Add/modify eslint rule - jest/valid-expect - to allow for 2nd optional message argument/

Possible solution:
While the 2nd optional argument for message works. Eslint's jest/valid-expect rule fires unless disabled above. Extend the current jest/valid-expect rule with additional value such as allow-message

@0b10
Copy link

0b10 commented Oct 6, 2019

@alee8 did you find a way to just disable the linting for additional properties only? I couldn't find a way in either of the eslint/jest-plugin docs - I just set the rule to "off" for now.

@G-Rath
Copy link

G-Rath commented May 16, 2020

@mattphillips this issue has been closed by jest-community/eslint-plugin-jest#584

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants