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

[meta] tune luacheck for norns #1

Open
2 tasks
pq opened this issue Jul 21, 2018 · 0 comments
Open
2 tasks

[meta] tune luacheck for norns #1

pq opened this issue Jul 21, 2018 · 0 comments
Labels
advise Having to do with scry advice meta A meta issue

Comments

@pq
Copy link
Owner

pq commented Jul 21, 2018

currently luacheck is not tuned for norns and (some) "idiomatic" scripts are very linty.

the reported issues fall into a some rough categories.

  1. legitimate
  2. legitimate but fussy
  3. bogus
  4. bogus but potentially legitimate

legitimate issues (1) are good as is but fussy ones (2) should probably be hidden by default w/ an opt-in. entirely bogus issues (3) should just get filtered out whereas we should strive to make the ones that are potentially valuable (4) legitimate.


follow-ups to address:

bogus lints

fussy lints

@pq pq added advise Having to do with scry advice meta A meta issue labels Jul 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
advise Having to do with scry advice meta A meta issue
Projects
None yet
Development

No branches or pull requests

1 participant