This repository was archived by the owner on Mar 23, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 509
isEmpty of null #1816
Labels
Comments
Thanks for the report - yeah please add a snippet to reproduce and your config. If you run with |
Config I'm using: {
"disallowEmptyBlocks": true,
"disallowMixedSpacesAndTabs": true,
"disallowSpaceBeforeComma": true,
"disallowSpaceBeforeSemicolon": true,
"disallowTrailingWhitespace": true,
"disallowTrailingComma": true,
"disallowSpacesInsideParentheses": true,
"requireCapitalizedConstructors": true,
"requireLineFeedAtFileEnd": true,
"requireSemicolons": true,
"requireSpaceBetweenArguments": true,
"disallowYodaConditions": [
"==",
"===",
"!=",
"!=="
],
"requireSpaceAfterKeywords": [
"do",
"for",
"if",
"else",
"switch",
"case",
"try",
"catch",
"void",
"while",
"with",
"return",
"typeof"
],
"excludeFiles": [
"path/to/file.js"
]
} Unfortunately,
|
Hm, i think i know what the issue is, we would need to release again though |
markelog
added a commit
to markelog/node-jscs
that referenced
this issue
Sep 28, 2015
markelog
added a commit
to markelog/node-jscs
that referenced
this issue
Sep 28, 2015
Ok shouldn't of closed that - @danielstjules can you check if that PR/commit (f12830a) fixes it? |
2.2.1 is released! Hopefully fixes it? |
That fixed it, thank you! :) |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
With 2.2.0, I'm getting:
However, fixing that reveals another issue:
Can't right now, but hopefully I can come up with a small example later tonight.
The text was updated successfully, but these errors were encountered: