-
Notifications
You must be signed in to change notification settings - Fork 12
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
Setup CI/tests #44
Comments
@Enigmage Are you working on this one? I am asking since you have created this issue and haven't specified if you are working or not. If you are not working, i would like to work on it. |
@lucifer0987 Yeah sure go ahead. I'm not expecting this one to be only assigned to myself. Anyone can work on adding tests as mentioned in #45. |
@lucifer0987 @Enigmage - The tests are independent to the CI, the CI will just run all the tests on that branch once a PR is made. |
The issue has been free for a couple of weeks now. May I work on the CI and tests? |
@paramsiddharth - before setting up the CI, please make a PR with the tests only. |
Opening this on the request of @ankingcodes . It would be helpful to have an automated CI system that can test all PRs for proper coding style (proper indentation and other python standards) and merge/reject them based on the results.
The text was updated successfully, but these errors were encountered: