Skip to content

Conversation

@A77AY
Copy link
Collaborator

@A77AY A77AY commented Mar 26, 2025

No description provided.

@A77AY A77AY changed the title EMP-323: Enhance NPM publish action and installation logic [major] EMP-323: Enhance NPM publish action and installation logic Mar 26, 2025
@A77AY A77AY changed the title [major] EMP-323: Enhance NPM publish action and installation logic EMP-323: Enhance NPM publish action and installation logic [major] Mar 26, 2025
@A77AY A77AY changed the title EMP-323: Enhance NPM publish action and installation logic [major] EMP-323: Enhance NPM publish action and installation logic #major Mar 26, 2025
@A77AY A77AY requested a review from Copilot March 27, 2025 08:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request enhances the NPM publishing action and refines the NodeJS setup action by improving versioning logic and installation commands.

  • Updated publish action inputs to support conditional PR labeling, custom version tags, and version bump skipping.
  • Updated setup action to use actions/setup-node@v4, change the default NodeJS version to "lts/*", and add a new step to conditionally run "npm ci" or "npm install" with a force option.

Reviewed Changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.

File Description
publish/action.yml Modified inputs for managing version bump and publish conditions.
setup/action.yml Updated NodeJS setup, default version, and added dependency installation.
Files not reviewed (2)
  • .github/workflows/configured.yml: Language not supported
  • package.json: Language not supported
Comments suppressed due to low confidence (1)

publish/action.yml:8

  • [nitpick] The input name 'npm-skip-version-up' with the description 'Version not up' may be unclear. Consider renaming it to something like 'npm-skip-version-bump' and updating the description to 'Skip version bump' for improved clarity.
npm-skip-version-up:

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@A77AY A77AY changed the title EMP-323: Enhance NPM publish action and installation logic #major EMP-323: Enhance NPM publish action and installation logic [major] Mar 27, 2025
@A77AY A77AY merged commit 5b98064 into master Mar 27, 2025
2 checks passed
@A77AY A77AY deleted the ft/bump-ver branch March 27, 2025 09:06
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.

3 participants