Skip to content

Replaces deprecated macos-13 runner with macos-15-intel runner#2555

Merged
galabovaa merged 1 commit into
ERGO-Code:masterfrom
merschformann:master
Oct 10, 2025
Merged

Replaces deprecated macos-13 runner with macos-15-intel runner#2555
galabovaa merged 1 commit into
ERGO-Code:masterfrom
merschformann:master

Conversation

@merschformann
Copy link
Copy Markdown
Contributor

Description

Github announced that they will be retiring the macos-13 runner soon (see here). The runner seems to be mainly used to get a darwin/x64 environment. A sensible replacement should be the new macos-15-intel runner.

I hope this makes sense as a replacement runner. See above link as well for other recommendations. Happy to further assist if helpful. ☺️

Changes

  • Replaces deprecated macos-13 runner with new macos-15-intel runner.

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.11%. Comparing base (4d4e98c) to head (ff250ac).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2555      +/-   ##
==========================================
- Coverage   79.12%   79.11%   -0.01%     
==========================================
  Files         346      346              
  Lines       84943    84943              
==========================================
- Hits        67207    67205       -2     
- Misses      17736    17738       +2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jajhall jajhall requested a review from galabovaa October 4, 2025 10:46
@galabovaa galabovaa mentioned this pull request Oct 8, 2025
@galabovaa galabovaa merged commit c780f58 into ERGO-Code:master Oct 10, 2025
125 of 134 checks passed
@galabovaa
Copy link
Copy Markdown
Contributor

Accidentally merged in master instead of latest

@merschformann
Copy link
Copy Markdown
Contributor Author

Accidentally merged in master instead of latest

Hey!

Let me know if I can assist with anything. I saw the tests fail before merge and wasn't sure whether it was related to my change or not. I didn't dig in further. But let me know if there are any ongoing issues. Happy to help. I think I can find some time in the upcoming days if needed. ☺️

cc @galabovaa

@galabovaa
Copy link
Copy Markdown
Contributor

Hi @merschformann!

Thank you, this is all good, I have fixed the tests and your update is already in the "latest" branch.

The only thing you should keep in mind for future PRs is to open them to the "latest" branch and not master, my mistake here was not spotting this before I merged it!

@merschformann
Copy link
Copy Markdown
Contributor Author

Ah, thanks for the heads up. I'll try to keep it in mind. Sorry for the inconvenience. 😬

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