Skip to content

Conversation

@bhazelton
Copy link
Member

@bhazelton bhazelton commented Nov 7, 2025

Update the CI to build against:

  • linux, mac and windows (currently not working, but will be fixed by removing the healpy dependency swap astropy-healpix for healpy #75)
  • the newest releases of our dependencies -- critical to make sure we're keeping up
  • the oldest allowed versions of our dependencies -- critical to make sure our dependency ranges are accurate.

Also updates the minimum python to 3.11 because the minimum pyuvdata requires at least python 3.11. This also required updating the uv.lock file.

I also added uploading to codecov. I strongly prefer this to the current "roll our own" approach, however this can only represent coverage from the CI runs, so coverage is rather low. This will improve as we reduce our test data volume and get more code covered by the ci-runnable tests. For now I didn't add the codecov badge to our readme since it doesn't represent the true code coverage.

Types of Changes

  • Bug Fixes
  • New Feature(s)
  • Breaking Changes
  • Documentation
  • Version Change
  • Translation from FHD
  • Build or CI Change

Changelog

Build/CI changes

  • added testing against newest dependency releases and minimum compatible dependencies.
  • added testing against mac and windows (the latter is not working yet)
  • added uploading of coverage info to codecov

Checklist

General PR Checklist

  • I have the read the contribution guide
  • Add all the above to the Changelog into the unreleased section

Build/CI

  • Added a badge for any new CI actions or setups
  • Existing badges remain unaffected, if not, document why in changelog

@codecov
Copy link

codecov bot commented Nov 7, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@bhazelton
Copy link
Member Author

@nicholebarry I think this is ready and can go in as is, it at least improves the structure for future runs.

@bhazelton bhazelton merged commit 62f8ac8 into main Nov 18, 2025
22 checks passed
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.

3 participants