Skip to content

Conversation

@PussInDasBoot
Copy link
Collaborator

@PussInDasBoot PussInDasBoot commented May 15, 2025

  • Github actions were failing as Python versions were no longer supported on the latest ubuntu build on actions
  • Update the Github actions to install Anaconda3-2021.11 and install all dependencies as per the linux script.
  • Cache these dependencies and other libraries required (openmpi-bin libopenmpi-dev libhdf5-dev) to speed up test runs in future)
  • Remove Travis for CI as hasn't been used for 5 years

I removed the converagerc file as it was only including some directories to look at coverage for and excluding many others. I think it would be best to check coverage for the entire project. As I start increasing the coverage, I can build a new coveragerc if we feel it is necessary.

There are a lot of commits, but once this PR is approved and we're happy, I will squash and merge them.

@PussInDasBoot PussInDasBoot force-pushed the PB-2295-github-actions branch from 68d5f24 to c6cdf98 Compare May 15, 2025 01:56
@PussInDasBoot PussInDasBoot force-pushed the PB-2295-github-actions branch from c592bd2 to 71371bf Compare May 15, 2025 06:11
@PussInDasBoot PussInDasBoot force-pushed the PB-2295-github-actions branch from dabd3b8 to 546fff7 Compare June 11, 2025 03:06
@PussInDasBoot PussInDasBoot force-pushed the PB-2295-github-actions branch from 6d46f52 to 5381c58 Compare June 11, 2025 04:40
@PussInDasBoot PussInDasBoot force-pushed the PB-2295-github-actions branch from 0d16d10 to a3349b4 Compare June 12, 2025 01:14
@PussInDasBoot PussInDasBoot force-pushed the PB-2295-github-actions branch from 5a89bc4 to c42e71d Compare June 12, 2025 02:30
@PussInDasBoot PussInDasBoot self-assigned this Jun 12, 2025
@PussInDasBoot PussInDasBoot marked this pull request as ready for review June 12, 2025 03:01
@PussInDasBoot PussInDasBoot requested a review from geojunky June 12, 2025 03:01
@PussInDasBoot PussInDasBoot force-pushed the PB-2295-github-actions branch from 20d51ba to 6b96adf Compare June 12, 2025 23:34
@codecov-commenter
Copy link

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 ☂️

Copy link
Collaborator

@geojunky geojunky left a comment

Choose a reason for hiding this comment

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

Hi Melyn, I've suggested a few minor tweaks. We're almost there.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please add these requirements to conda_env_linux.sh as the final set of pip installs, so we can remove this requirements.txt altogether.

@PussInDasBoot PussInDasBoot requested a review from geojunky June 13, 2025 01:14
@geojunky geojunky merged commit f395124 into develop Jun 13, 2025
1 check passed
@PussInDasBoot PussInDasBoot deleted the PB-2295-github-actions branch June 16, 2025 02:28
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.

4 participants