Skip to content

MLE-31583 Create npm-publish-packages.yml - #1122

Merged
rjrudin merged 2 commits into
masterfrom
npm-publish-github-action
Sep 8, 2026
Merged

MLE-31583 Create npm-publish-packages.yml#1122
rjrudin merged 2 commits into
masterfrom
npm-publish-github-action

Conversation

@rjdew-progress

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The workflow currently contains misleading publishing documentation and skips all tests prior to publishing, increasing the risk of releasing broken artifacts.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds a GitHub Actions workflow to build and publish the marklogic npm package automatically when a GitHub Release is published.

Changes:

  • Introduces a new release-triggered workflow that checks out the code, sets up Node.js, installs dependencies, builds, and runs npm publish.
  • Configures the workflow for npm registry publishing and OIDC token permissions.
File summaries
File Description
.github/workflows/npm-publish-packages.yml New release workflow to build and publish the npm package.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 3
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/npm-publish-packages.yml Outdated
Comment thread .github/workflows/npm-publish-packages.yml Outdated
Comment thread .github/workflows/npm-publish-packages.yml
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@rjrudin
rjrudin merged commit c46ba81 into master Sep 8, 2026
4 checks passed
@rjdew-progress
rjdew-progress deleted the npm-publish-github-action branch September 8, 2026 23:55
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.

4 participants