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 the MacOs runner in our CI #396

Merged

Conversation

schroedtert
Copy link
Collaborator

As the MacOS 12 GitHub Action runner is no longer supported, we use more up-to-date versions in our CI. Also the version of the Ubuntu runner is now specified to get a better control, on what is used here.

Copy link

codecov bot commented Dec 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.22%. Comparing base (0fd1969) to head (089e5be).
Report is 2 commits behind head on main.

@@           Coverage Diff           @@
##             main     #396   +/-   ##
=======================================
  Coverage   63.22%   63.22%           
=======================================
  Files          14       14           
  Lines        1357     1357           
=======================================
  Hits          858      858           
  Misses        499      499           

As the MacOS 12 GitHub Action runner is no longer supported, we use more
up-to-date versions in our CI.
Instead of using the ubuntu-latest runner, use the specifc version
24.04. This allows better control and is not updated, without our
knowledge.
@schroedtert schroedtert merged commit 871cb64 into PedestrianDynamics:main Dec 7, 2024
25 checks passed
@schroedtert schroedtert deleted the update-macos-runner branch December 7, 2024 10:09
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.

1 participant