search

cflint / CFLint

Static code analysis for CFML (a linter)
BSD 3-Clause "New" or "Revised" License
174 stars 84 forks source link

cflint ignore:line support for cfscript #741

Open denuno opened 1 month ago

denuno commented 1 month ago

I should have looked closer, as I thought this was an existing test that was failing, but it looks like it was something I was in the process of addressing and never committed.

denuno commented 1 month ago

Oh, this test I had was actually about an empty config I guess, tho there was a bug with ignore:line as well… hmm

denuno commented 1 month ago

was this the issue you meant to ask that on?

It looks like pre is not in the list of "phrasing content" so I guess it is invalid inside a p tag

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/p#technical_summary