-
Notifications
You must be signed in to change notification settings - Fork 3
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 a linter check with github actions #36
Comments
Hey @Pradhvan, I am new to open source and will like to contribute to this project, can you guide me on what needs to be done. |
Hi @slayer321 there is no GitHub action running with the repo. So it would be good if you could add a GitHub action that checks linting errors when a new commit. This could also be used when a PR is made from any branch to master and on a successful merge to master. You can take a reference from this post ignore the step with poetry and tests. The general idea would be the same. |
Hey @Pradhvan , thank you for adding the blog , I read it and also did my own research and I come up with this workflow gist . This is the first time I'm working with |
@slayer321 yes, something similar to this. You can create a branch and add it to the GitHub action. Test if that works and create a PR if everything works out. Thank you for helping out. 😄 |
The text was updated successfully, but these errors were encountered: