-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
new-line-before-expect
not ignoring earlier expect
#170
Comments
I see this bug too, with multiline expect statements like these two times:
eslint-plugin-jasmine: 4.1.1 |
Probably same reason as #161, everything that is not a line starting with |
Issue 161 states that fix was released in version 4.1.3. I am using that version and the issue described here definitely remains. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Do you want to request a feature or report a bug?
Bug
What is the current behavior?
The second expect raises a
new-line-before-expect
warning:If the current behavior is a bug, please provide the steps to reproduce.
Code above.
What is the expected behavior?
Based on this documentation (also common sense): https://github.com/tlvince/eslint-plugin-jasmine/blob/master/docs/rules/new-line-before-expect.md (last example) no warning.
Please mention your node.js, eslint-plugin-jasmine and operating system version.
node: v10.5.0
eslint-plugin-jasmine: 2.10.1
MacOS: 10.12.6
The text was updated successfully, but these errors were encountered: