-
Notifications
You must be signed in to change notification settings - Fork 956
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
Fix the CI #246
Comments
I'm a big fan of CI issues and would love to help out here. Are you still planning to use Travis CI, or are you interested in moving to GitHub Actions? I'm more familiar with the later, but interested nonetheless - would just need some more info to get caught up! |
Sure, feel free to take this and use GitHub Actions. Note that PySnooper is pretty mature and there isn't a lot of development happening on it anymore, so if you're taking on this issue, it's mostly for your own fun. |
That sounds good, is there any specific tooling you want in the process? Otherwise I'll add from what I typically use, but want to make sure anything you previously were set up for gets included. |
Your choice. As long as the tests get run. |
We used to have Travis running tests automatically on PRs. It doesn't do that anymore. We should fix that.
The text was updated successfully, but these errors were encountered: