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

CodeSniffer says JS exceeding 120 characters? #30

Open
danvim opened this issue Mar 29, 2016 · 1 comment
Open

CodeSniffer says JS exceeding 120 characters? #30

danvim opened this issue Mar 29, 2016 · 1 comment

Comments

@danvim
Copy link

danvim commented Mar 29, 2016

First I have to say I don't think this is a problem created by the plugin, but is CodeSniffer supposed to check JS?

The error is Line exceeds 120 characters. I don't regard this as a JS problem and CodeSniffer should have nothing to do with JS because it is very easy to exceed the limit in JS either by using strings or JQuery.

I think there should be an option to turn off validating for <script> tags. Where should I report this problem?

@mikaeljorhult
Copy link
Owner

To my knowledge there's no way to do this in Code Sniffer. I've never tried to use it on files containing anything else than PHP only.

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

No branches or pull requests

2 participants