-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add Linting Step/Check in Travis CI #9
Comments
A suggestion for this |
Yeah, this seems to be the recommended course of action. I'll look into it |
Precommit is a good method to check for lint errors but CI is also needed. I would recommend using github actions for this instead of Travis CI |
I can work on this if you can assign me :p |
@RohanJnr Sure, I'll assign you. Why Github Actions over Travis CI though? Just want to weigh the pros and cons 😄 |
github actions is simply available on github, its one of their products so why go for a third party CI tool like travis |
while that would simply be more convenient, I'd just like to know if we're missing out on anything in terms of features by choosing github actions as opposed to Travis. I'd appreciate any more information on this :D |
there are a bunch of resources online about that and actions is mostly the choice for most application. you should be able to accomplish the same using both. |
I would suggest flake8 as a linting tool. Since you're using it with black you'll have to create a compatible configuration. (ref) |
If you're going to use a 3rd party CI, I wouldn't recommend Travis, there were a lot of weird issues which popped up for us while using Travis, go with CircleCI or Jenkins |
GitHub actions? |
that's pretty much it. Check for code formatting using
black
, along with a linting tool that is the best choice.The text was updated successfully, but these errors were encountered: