Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dinmukhamedm
Copy link
Member

@dinmukhamedm dinmukhamedm commented Jul 1, 2025

pnpm checks that you are publishing from main/master. This is good on its own, but breaks when publishing from a tag (github release). Related: pnpm/pnpm#5894


Important

Add --no-git-checks to pnpm publish in GitHub Actions to allow publishing from tags.

  • Behavior:
    • Adds --no-git-checks flag to pnpm publish command in .github/workflows/publish-to-npm.yml.
    • Allows publishing from a tag, bypassing the default branch check (main/master).
  • Context:
    • Addresses issue where pnpm fails to publish from a GitHub release tag due to branch checks.

This description was created by Ellipsis for f6c8e19. You can customize this summary. It will automatically update as commits are pushed.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Caution

Changes requested ❌

Reviewed everything up to f6c8e19 in 1 minute and 11 seconds. Click for details.
  • Reviewed 12 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

Workflow ID: wflow_RCe1Y8GCp2blcSNh

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@dinmukhamedm dinmukhamedm merged commit 3460ec7 into main Jul 1, 2025
3 checks passed
@dinmukhamedm dinmukhamedm deleted the fix/publish-action branch July 1, 2025 12:11
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