Skip to content
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

Update CI #583

Merged
merged 2 commits into from
Nov 27, 2024
Merged

Update CI #583

merged 2 commits into from
Nov 27, 2024

Conversation

molpopgen
Copy link
Collaborator

  • bump ubuntu image and python
  • Remove version pins from requirements files
  • Add requirements_CI.txt
  • update remaining work flow files
  • remove global locked file
  • Add pinned requirements files
  • use pinned requirements files in CI

@molpopgen
Copy link
Collaborator Author

@grahamgower -- This is all working other than codecov. The one test that needed updating was pretty straightforward as there was already a pytest warning filter in place. I just made the filter much stricter.

I think I should also squash this down to 1 commit before merging?

Copy link
Member

@grahamgower grahamgower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. It would be good to squash this at least a little, yeah. Feel free to merge when you're satisfied.

@grahamgower
Copy link
Member

@molpopgen, If you rebase, then the codecov upload should now work.

Do we also need to bump the .github/mergify.yml file for dependabot to work again? Do you know if/how dependabot integrates with the uv method of managing requirements?

@molpopgen
Copy link
Collaborator Author

Do we also need to bump the .github/mergify.yml file for dependabot to work again? Do you know if/how dependabot integrates with the uv method of managing requirements?

I'll look into this. I seem to recall that github has opened an issue on this repo for one or more of our automated things.

Copy link

codecov bot commented Nov 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.81%. Comparing base (a296850) to head (109392f).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #583   +/-   ##
=======================================
  Coverage   99.81%   99.81%           
=======================================
  Files           5        5           
  Lines        1600     1600           
=======================================
  Hits         1597     1597           
  Misses          3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@molpopgen
Copy link
Collaborator Author

@grahamgower -- I squashed all previous commits and separately updated mergify.

As for uv, it only handles stuff in the Python ecosystem. AFAICT, it has no understanding of github work flows, etc..

@molpopgen
Copy link
Collaborator Author

@grahamgower -- I was hoping to have some way to actually test mergify but I have no idea how it works. I'm trying to find some kind of @mergify squash and rebase command but not having any luck with their docs.

@grahamgower
Copy link
Member

I think it's one of those things that is only acted upon after it's committed in the repository.

@grahamgower grahamgower merged commit 9b5c5a5 into main Nov 27, 2024
13 checks passed
@molpopgen molpopgen deleted the bump_python_CI branch November 27, 2024 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants