8000 docs: streamline the release PR merge steps in the releasing skill by morellodev · Pull Request #329 · awesome-reveal/awesome-reveal · GitHub
[go: up one dir, main page]

Skip to content

docs: streamline the release PR merge steps in the releasing skill - #329

Merged
morellodev merged 1 commit into
nextfrom
docs/releasing-skill-retro
Oct 18, 2026
Merged

morellodev merged 1 commit into
nextfrom
docs/releasing-skill-retro

Conversation

@morellodev
Copy link
Copy Markdown
Collaborator

Summary

This PR changes the "Merge a release PR" steps of the releasing-packages skill. The changes come from the merge of #328:

  • Step 2 no longer runs npm trust list. The agent shell has no npm login, so that command fails with E401. The publish job publishes only through OIDC, so a package that it released before has a trusted publisher. The step now uses npm view to find new packages.
  • Step 3 watches the checks without output and merges only when they pass. It also deletes the branch.
  • Step 4 runs gh run watch in the background. The publish job can wait 20 minutes until npm shows each version, which is longer than the agent's foreground command limit.
  • Step 5 checks only the dist-tags, because a green run already proves that npm shows each version.

Checklist

  • The PR targets the next branch.
  • The PR title follows Conventional Commits. A breaking change has ! and a BREAKING CHANGE: footer.
  • Tests cover the change. Does not apply: the change edits only an agent skill doc.
  • A change to an option, a default or an export also updates the READMEs, the consumer skills and the examples. Does not apply: no option, default or export changes.

@morellodev
morellodev merged commit 37c4afb into next Oct 18, 2026
19 checks passed
@morellodev
morellodev deleted the docs/releasing-skill-retro branch October 18, 2026 13:53
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

0