Skip to content
This repository has been archived by the owner on Feb 7, 2024. It is now read-only.

[Innovation] Add ARM64 MacOS runner #149

Merged
merged 2 commits into from
Feb 1, 2024
Merged

[Innovation] Add ARM64 MacOS runner #149

merged 2 commits into from
Feb 1, 2024

Conversation

awharn
Copy link
Member

@awharn awharn commented Jan 31, 2024

What It Does

Adds macos-14 to the test strategy, validating the CLI and Daemon function as expected on ARM64.

How to Test

Review Checklist
I certify that I have:

Additional Comments

Signed-off-by: Andrew W. Harn <andrew.harn@broadcom.com>
Copy link

codecov bot commented Jan 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7fd4dad) 93.28% compared to head (cfe31c7) 93.28%.

❗ Current head cfe31c7 differs from pull request most recent head b69321f. Consider uploading reports for the commit b69321f to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #149   +/-   ##
=======================================
  Coverage   93.28%   93.28%           
=======================================
  Files          74       74           
  Lines         760      760           
  Branches       94       94           
=======================================
  Hits          709      709           
  Misses         51       51           

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

Signed-off-by: Andrew W. Harn <andrew.harn@broadcom.com>
Copy link

sonarcloud bot commented Jan 31, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@awharn awharn merged commit f5ddff9 into master Feb 1, 2024
17 checks passed
@awharn awharn deleted the add-arm-mac-runner branch February 1, 2024 21:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Closed
Development

Successfully merging this pull request may close these issues.

4 participants