Skip to content

Wait for a local npm publish instead of racing it in publish.yml - #63

Merged
elkaix merged 1 commit into
mainfrom
ci/publish-wait-local
Sep 24, 2026
Merged

elkaix merged 1 commit into
mainfrom
ci/publish-wait-local

Conversation

@elkaix

@elkaix elkaix commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

npm processes uploads asynchronously (PUT 202), so the tag workflow saw
v0.18.0 and v0.18.1 as unpublished and failed with ENEEDAUTH, skipping
the registry and GitHub release jobs until a manual rerun. Without an
NPM_TOKEN, poll for the version for up to 10 minutes.

npm processes uploads asynchronously (PUT 202), so the tag workflow saw
v0.18.0 and v0.18.1 as unpublished and failed with ENEEDAUTH, skipping
the registry and GitHub release jobs until a manual rerun. Without an
NPM_TOKEN, poll for the version for up to 10 minutes.
@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 7 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 4 included reviews currently available. Your 68 included PR review attempts over the past 7 days set your current allowance at 4 reviews per hour.

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 1a1dab97-75f7-4288-b4b7-2ebed7e3ec38

📥 Commits

Reviewing files that changed from the base of the PR and between 7211488 and cc68631.

📒 Files selected for processing (1)
  • .github/workflows/publish.yml

Comment @coderabbitai help to get the list of available commands.

@elkaix
elkaix merged commit dcad79a into main Sep 24, 2026
8 checks passed
@elkaix
elkaix deleted the ci/publish-wait-local branch September 24, 2026 19:22
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