Skip to content
This repository has been archived by the owner on Apr 6, 2022. It is now read-only.

Add checker for python code #20

Closed
nashif opened this issue Feb 23, 2019 · 3 comments
Closed

Add checker for python code #20

nashif opened this issue Feb 23, 2019 · 3 comments
Labels
enhancement New feature or request

Comments

@nashif
Copy link
Member

nashif commented Feb 23, 2019

Check for python syntax using one of the many linters:

  • flake8
  • pylint
  • ..

preferably this should work on diffs rather on complete files.

@nashif nashif added the enhancement New feature or request label Feb 23, 2019
@marc-hb
Copy link
Collaborator

marc-hb commented Mar 19, 2019

BTW here's pylint's most missing feature, especially in the context of focusing on new code and ignoring warnings on existing code:

usefulpylint() { pylint "$@" | sort -t ":" -k 2,2n ; }
usefulpylint -E scripts/sanitycheck

@ulfalizer
Copy link
Collaborator

Only takes ~5 seconds or so run pylint3 on all .py files in the tree, so for stuff that should always generate a failure, it's easy to do that instead.

@marc-hb
Copy link
Collaborator

marc-hb commented Mar 20, 2019

Duplicate of issue zephyrproject-rtos/zephyr#10338 ?

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

No branches or pull requests

3 participants