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

[Backport 2.x] Enhance darwin-arm64 release #5466

Closed
wants to merge 2 commits into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 89dc051 from #5455.

* Add min-builds for `darwin-x64` and `darwin-arm64`.
* Expose `--darwin-arm` as a parameter of the build script.
* Work around build's cleanup failure when removal of an already deleted folder is attempted.
* Exclude bundling fallbacks with `darwin-arm64` since Node.js v14 has no releases for them.
* Update tests to accommodate `darwin-arm64`.
* Generalize the Node.js versions used in some tests.

Signed-off-by: Miki <[email protected]>
(cherry picked from commit 89dc051)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot added the autocut Skip the changelog verification check on backports label Nov 10, 2023
@opensearch-trigger-bot opensearch-trigger-bot bot added Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry distinguished-contributor labels Nov 10, 2023
Copy link

codecov bot commented Nov 10, 2023

Codecov Report

❗ No coverage uploaded for pull request base (2.x@ba20c9f). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             2.x    #5466   +/-   ##
======================================
  Coverage       ?   61.92%           
======================================
  Files          ?     2847           
  Lines          ?    55473           
  Branches       ?     8881           
======================================
  Hits           ?    34354           
  Misses         ?    19189           
  Partials       ?     1930           
Flag Coverage Δ
Linux_2 55.09% <ø> (?)
Windows_1 35.27% <ø> (?)
Windows_3 43.85% <ø> (?)

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

Copy link
Collaborator

@AMoo-Miki AMoo-Miki left a comment

Choose a reason for hiding this comment

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

Something doesn’t add up. I will manually backport.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut Skip the changelog verification check on backports distinguished-contributor Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry v2.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants