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

Node rc fixes + unpublish on failure mechanism #2737

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

avifenesh
Copy link
Member

@avifenesh avifenesh commented Nov 21, 2024

This pull request includes several updates to the build and deployment workflows, as well as dependency updates. The most important changes are improvements to the npm publish process, the addition of a package deprecation step on failure, and updates to the build matrix and dependencies.

Workflow improvements:

  • .github/workflows/npm-cd.yml: Improved the npm publish process by enhancing error handling and logging, and simplified the command to filter out notices and warnings.
  • .github/workflows/npm-cd.yml: Added a step to deprecate packages if the job fails, ensuring deprecated packages are marked correctly. This includes installing jq and setting the RELEASE_VERSION based on environment variables.

Build matrix updates:

Dependency updates:

  • node/package.json: Updated typescript to version ^5.6.3 and uuid to version ^11.0.3 to ensure compatibility and leverage the latest features and fixes.

Issue link

This Pull Request is linked to issue (#2735)

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.

@avifenesh avifenesh added this to the 1.2 milestone Nov 21, 2024
@avifenesh avifenesh self-assigned this Nov 21, 2024
@avifenesh avifenesh marked this pull request as ready for review November 21, 2024 23:48
@avifenesh avifenesh requested a review from a team as a code owner November 21, 2024 23:48
python/README.md Outdated Show resolved Hide resolved
.github/json_matrices/supported-languages-versions.json Outdated Show resolved Hide resolved
@avifenesh avifenesh force-pushed the node-rc branch 2 times, most recently from 03709bd to 8a8be2e Compare November 22, 2024 11:00
@avifenesh avifenesh force-pushed the node-rc branch 2 times, most recently from 0b53d46 to cd9f112 Compare November 22, 2024 12:02
@avifenesh avifenesh merged commit 88eca43 into valkey-io:release-1.2 Nov 25, 2024
70 of 72 checks passed
prateek-kumar-improving pushed a commit that referenced this pull request Nov 26, 2024
* Update Node.js Docker image and enhance npm publish workflow with unpublish feature

Signed-off-by: avifenesh <[email protected]>

* unpublishing a package owned by more than one cant be unpublised, hence the usage is deprecating

Signed-off-by: avifenesh <[email protected]>

---------

Signed-off-by: avifenesh <[email protected]>
@avifenesh avifenesh deleted the node-rc branch February 9, 2025 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD CI/CD related
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants