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

ci(agent): Add macOS ARM64 to AutoGPT Python CI matrix #7041

Merged
merged 2 commits into from
Mar 22, 2024

Conversation

Pwuts
Copy link
Member

@Pwuts Pwuts commented Mar 22, 2024

  • Add macos-arm64 to platform-os matrix, and map it to macos-14 runner

Background

When it comes to testing, macOS != macOS on M1 because it requires different distributions of platform-specific packages.

PR Quality Scorecard ✨

  • Have you used the PR description template?   +2 pts
  • Is your pull request atomic, focusing on a single change?   +5 pts
  • Have you linked the GitHub issue(s) that this PR addresses?   +5 pts
  • Have you documented your changes clearly and comprehensively?   +5 pts
  • Have you changed or added a feature?   -4 pts
    • Have you added/updated corresponding documentation?   +4 pts
    • Have you added/updated corresponding integration tests?   +5 pts
  • Have you changed the behavior of AutoGPT?   -5 pts
    • Have you also run agbenchmark to verify that these changes do not regress performance?   +10 pts

@Pwuts Pwuts requested a review from a team as a code owner March 22, 2024 12:41
Copy link

netlify bot commented Mar 22, 2024

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit be9d927
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/65fd84e592a8220008aff6b0

Copy link

codecov bot commented Mar 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.51%. Comparing base (2a0e087) to head (be9d927).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7041   +/-   ##
=======================================
  Coverage   45.51%   45.51%           
=======================================
  Files         139      139           
  Lines        6530     6530           
  Branches      917      917           
=======================================
  Hits         2972     2972           
  Misses       3408     3408           
  Partials      150      150           
Flag Coverage Δ
Linux 45.40% <ø> (ø)
Windows 43.64% <ø> (ø)
autogpt-agent 45.48% <ø> (ø)
macOS 44.85% <ø> (ø)

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

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

@Pwuts Pwuts force-pushed the ci-agent/add-macos-arm64 branch 2 times, most recently from 77a6469 to b6f7a6c Compare March 22, 2024 13:17
@Pwuts Pwuts merged commit 30bc761 into master Mar 22, 2024
17 checks passed
@Pwuts Pwuts deleted the ci-agent/add-macos-arm64 branch March 22, 2024 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant