Skip to content

Add macos support - #318

Merged
sloede merged 20 commits into
mainfrom
bg/add-macos
Sep 29, 2026
Merged

sloede merged 20 commits into
mainfrom
bg/add-macos

Conversation

@benegee

@benegee benegee commented Sep 28, 2026 •

Copy link
Copy Markdown
Collaborator

This PR fully integrates macos in our workflow.

Supersedes #302 #298

@codecov

codecov Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.11%. Comparing base (854d0bc) to head (a93b6dc).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #318   +/-   ##
=======================================
  Coverage   96.11%   96.11%           
=======================================
  Files          25       25           
  Lines        1260     1260           
  Branches       74       74           
=======================================
  Hits         1211     1211           
  Misses         45       45           
  Partials        4        4           
Flag Coverage Δ
unittests 96.11% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

@benegee
benegee marked this pull request as ready for review September 28, 2026 14:02
@benegee
benegee requested a review from sloede September 28, 2026 14:16
sloede
sloede previously approved these changes Sep 28, 2026

@sloede sloede left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@benegee

benegee commented Sep 29, 2026

Copy link
Copy Markdown
Collaborator Author

I reused the package compiler test step as macos smoke tests, added a Fortran example, and realized the package compiler build was not using t8code. I changed that and it just worked.

Would you re-approve @sloede ?

@sloede
sloede merged commit 9a350ac into main Sep 29, 2026
20 of 22 checks passed
@sloede
sloede deleted the bg/add-macos branch September 29, 2026 14:18
This was referenced Sep 29, 2026
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