-
Notifications
You must be signed in to change notification settings - Fork 28
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
cache CI environment #280
base: main
Are you sure you want to change the base?
cache CI environment #280
Conversation
@stscieisenhamer @Witchblade101 are the differences in test success per-Python-version a concern? |
@zacharyburnett Yes. @mfixstsci plans on investigating this. |
a99e727
to
e288138
Compare
flake8
checksruff
checks
4df9bd4
to
c66d882
Compare
ruff might be a step too extreme... but ok. 😸 |
ruff
checks
Good point, it does ask for a lot of changes; I'll take it out |
@zacharyburnett should we remove python 3.7 and 3.8 from the testing matrix? Maybe add 3.12 if that's an option! |
Alright, I took out |
Nice! Thank you @zacharyburnett. We can leave 3.12 out for now, I didn't know how far along it was in development. We can add it once there is a more stable version available. I don't know what it was about python 3.8 and causing our testing matrix to fail. It was the only version that has given us any trouble... |
This PR does the following:
add.flake8
for Flake8 configurationapply formatting to codebase to satisfyruff
checks