Skip to content

Add publish workflow#8

Merged
salvador-barboza merged 1 commit intomainfrom
add-workflow
May 6, 2025
Merged

Add publish workflow#8
salvador-barboza merged 1 commit intomainfrom
add-workflow

Conversation

@salvador-barboza
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings May 6, 2025 01:00
@salvador-barboza salvador-barboza merged commit e4db859 into main May 6, 2025
3 checks passed
@salvador-barboza salvador-barboza deleted the add-workflow branch May 6, 2025 01:00
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 PR introduces a new GitHub Actions workflow to publish releases to the npm registry whenever a release is created or manually dispatched.

  • Adds a publish.yml workflow triggered on release creation and workflow_dispatch
  • Checks out repository code and sets up Node.js v20 with the npm registry

uses: actions/setup-node@v3
with:
node-version: '20'
registry-url: 'https://registry.npmjs.org'
Copy link

Copilot AI May 6, 2025

Choose a reason for hiding this comment

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

The workflow is missing steps to authenticate with npm and actually publish the package. Consider adding a step to configure the npm auth token (e.g., npm config set //registry.npmjs.org/:_authToken ${{ secrets.NPM_TOKEN }}) and a step to run npm publish.

Suggested change
registry-url: 'https://registry.npmjs.org'
registry-url: 'https://registry.npmjs.org'
- name: Authenticate with npm
run: echo "//registry.npmjs.org/:_authToken=${{ secrets.NPM_TOKEN }}" > ~/.npmrc
- name: Publish package
run: npm publish

Copilot uses AI. Check for mistakes.
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