Skip to content

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Jan 31, 2024

Today GitHub introduced the new M1 runners, making it possible to build macos-arm64 wheels without cross-building.

Remove old hacked codes for cross-building.

@codecov
Copy link

codecov bot commented Jan 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b800043) 74.22% compared to head (826071e) 74.32%.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #3206      +/-   ##
==========================================
+ Coverage   74.22%   74.32%   +0.09%     
==========================================
  Files         313      344      +31     
  Lines       27343    31873    +4530     
  Branches      908     1592     +684     
==========================================
+ Hits        20296    23690    +3394     
- Misses       6510     7258     +748     
- Partials      537      925     +388     

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

@njzjz njzjz requested a review from wanghan-iapcm January 31, 2024 00:49
@wanghan-iapcm wanghan-iapcm merged commit 664c70b into deepmodeling:devel Jan 31, 2024
njzjz added a commit to njzjz/deepmd-kit that referenced this pull request Feb 4, 2024
Today [GitHub introduced the new M1
runners](https://github.blog/changelog/2024-01-30-github-actions-introducing-the-new-m1-macos-runner-available-to-open-source/),
making it possible to build macos-arm64 wheels without cross-building.

Remove old hacked codes for cross-building.

(cherry picked from commit 664c70b)
wanghan-iapcm pushed a commit that referenced this pull request Feb 4, 2024
Today [GitHub introduced the new M1
runners](https://github.blog/changelog/2024-01-30-github-actions-introducing-the-new-m1-macos-runner-available-to-open-source/),
making it possible to build macos-arm64 wheels without cross-building.

Remove old hacked codes for cross-building.

(cherry picked from commit 664c70b)
@njzjz njzjz mentioned this pull request Apr 2, 2024
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