Add Pylint GitHub Action#76
Conversation
Green-Avocado
left a comment
There was a problem hiding this comment.
See pr joseph784#2
fixes bug actions/starter-workflows#636
Co-authored-by: Jason Ngo <greenavocado@protonmail.com>
SilentSys
left a comment
There was a problem hiding this comment.
Looks good. So is this going to start working once it gets merged into master or do we need to set up an account or give something access to this repo?
Also wondering if it would be worth consolidating this into CircleCI so that we don't have to maintain two different CI services.
Green-Avocado
left a comment
There was a problem hiding this comment.
Avoid running pylint on unchanged files
Co-authored-by: Jason Ngo <greenavocado@protonmail.com>
Co-authored-by: Jason Ngo <greenavocado@protonmail.com>
Co-authored-by: Andrei Popescu <andy.pope.13@gmail.com>
Co-authored-by: Jason Ngo <greenavocado@protonmail.com>
No setup is required, it should be good to go once it's in main
I think this would be easier just doing it this way, but we could consolidate it if you want |
No description provided.