MISSING_SEMI not detecting missing semicolons

cflint / CFLint

Static code analysis for CFML (a linter)

BSD 3-Clause "New" or "Revised" License

174 stars 84 forks source link

MISSING_SEMI not detecting missing semicolons #582

Open KamasamaK opened 6 years ago

KamasamaK commented 6 years ago

I have removed semicolons in a component body, function body, and within <cfscript> tags in a tag file and none of them are reporting MISSING_SEMI. I do not have that rule excluded in my .cflintrc. The only way I could get it to report is via #580, which is a false positive.

EDIT: Testing on the latest dev build with ad7e24f as the latest commit.

KamasamaK commented 6 years ago

I've also noticed that it is not documented in the cflint.definition.json, cflint.description.txt, or RULES.md. Is this not a supported rule?

KamasamaK commented 6 years ago

I was able to get it to trigger properly in a very specific situation.

Triggers: foo.append(bar)

Does not trigger: testVar = foo.append(bar)

arrayAppend(foo, bar)

devalnor commented 5 years ago

Any solution for this?

blangsmgi commented 5 years ago

I am also having this issue. Any updates?