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

Rewrite linter and add support for linting configuration #535

Open
6 tasks
Zethson opened this issue Dec 3, 2020 · 1 comment
Open
6 tasks

Rewrite linter and add support for linting configuration #535

Zethson opened this issue Dec 3, 2020 · 1 comment
Labels
enhancement New feature or request

Comments

@Zethson
Copy link
Member

Zethson commented Dec 3, 2020

  • We should refactor our linter so that we are not using a metaclass for every linter, which gets all in the python script defined linting classes, but so that the template linter just gets a list of imported linting functions and calls them. Those should then be more generalized
  • Our linter should be configurable. I envision that the cookietemple.cfg file has a section [linting] where one can disable checks
[linting]
general-7
cli-python-2
  • cookietemple lint should output a json or yaml file describing a summary of the linting results
  • the pip package outdated requests should be parallelized Run linting requests async #494
  • magic numbers should be removed in the check_section function
  • Advanced: Improve efficiency of the linting by applying several linting functions while reading a single file
@Imipenem

This comment has been minimized.

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

No branches or pull requests

2 participants