Skip to content

NPM publish workflow now triggers only on published releases.#33

Merged
mehmet-kozan merged 2 commits intomainfrom
ci-publish
Oct 19, 2025
Merged

NPM publish workflow now triggers only on published releases.#33
mehmet-kozan merged 2 commits intomainfrom
ci-publish

Conversation

@mehmet-kozan
Copy link
Copy Markdown
Owner

This pull request updates the workflow for publishing the NPM package and makes a minor documentation change. The main focus is on simplifying and improving the GitHub Actions workflow to ensure publishing only happens when a release is published, and to streamline the testing and publishing steps.

Workflow improvements:

  • The NPM publish workflow now triggers only on published releases, not on version tags, making the release process more intentional and reliable.
  • The integration test job for multiple Node.js versions was removed, simplifying the workflow and reducing CI run time. Now, only unit tests are run before publishing.
  • The logic for extracting the version tag was simplified to only handle release events, removing fallback handling for tag pushes.
  • The publish step now sets the NPM tag to beta if the version contains "beta", otherwise uses latest, ensuring correct tagging for pre-releases and stable versions. The publish command also explicitly sets public access.

Documentation update:

  • The CLI documentation for the --magic option was updated to remove the default value, clarifying its usage.

Simplifies the GitHub Actions workflow by removing integration tests, only triggering on release publication, and improving npm publish tagging logic. Also updates CLI documentation to clarify the '--magic' option.
@mehmet-kozan mehmet-kozan self-assigned this Oct 19, 2025
@mehmet-kozan mehmet-kozan merged commit 69eec3e into main Oct 19, 2025
11 checks passed
@mehmet-kozan mehmet-kozan deleted the ci-publish branch October 20, 2025 00:41
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.

1 participant