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 main] remove native package references for MacOS x64 architecture #2808

Merged

Conversation

jbrinkman
Copy link
Collaborator

Issue link

This Pull Request resolves issue (#2799).

This change removes the native package reference in package.json and updates the npm/glide index file to remove the native binding for the Darwin-x64 package which was removed in the Valkey-Glide 1.2.0 release.

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one issue.
  • Commit message has a detailed description of what changed and why.
  • Tests are added or updated.
  • CHANGELOG.md and documentation files are updated.
  • Destination branch is correct - main or release
  • Commits will be squashed upon merging.

@jbrinkman jbrinkman requested a review from a team as a code owner December 12, 2024 13:35
@yipin-chen
Copy link
Collaborator

CI is failing on DCO.

@jbrinkman jbrinkman force-pushed the node/jbrinkman-remove-darwin-x64 branch from 1bed431 to 7b1b005 Compare December 13, 2024 21:45
@jamesx-improving jamesx-improving merged commit 11e339b into valkey-io:main Dec 13, 2024
19 checks passed
@jamesx-improving jamesx-improving deleted the node/jbrinkman-remove-darwin-x64 branch December 13, 2024 22:34
@jbrinkman jbrinkman restored the node/jbrinkman-remove-darwin-x64 branch December 16, 2024 18:04
@jbrinkman jbrinkman changed the title remove native package references for MacOS x64 architecture [backport main] remove native package references for MacOS x64 architecture Dec 16, 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.

5 participants