-
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
2024 fixings #164
2024 fixings #164
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from the checks not passing, and the <python=3.7 questions, lgtm. Also 👋 hey :)
Welcome to Codecov 🎉Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment. Thanks for integrating Codecov - We've got you covered ☂️ |
Ok, I think this is finally good to go. There are some other modernization steps needed now (eg removing six dependency), but at least the CI runs and we're in the green. |
Remove imp
Modernize setup script
Migrate from travis to gh actions