Skip to content

feat(release-notes): dry-run#21

Merged
adrmachado-public merged 5 commits into
mainfrom
feat/dry-run-typescript
Jun 8, 2026
Merged

feat(release-notes): dry-run#21
adrmachado-public merged 5 commits into
mainfrom
feat/dry-run-typescript

Conversation

@adrmachado-public

Copy link
Copy Markdown
Contributor

Two fixes to actions/release-notes/action.yml:

  • add explicit actions/setup-node@v6 step — the action previously assumed Node.js was already set up by a prior action in the same job, creating a fragile implicit dependency
  • Replace awk/heredoc with TypeScript — the CHANGELOG parsing step used a multi-line awk expression writing to GITHUB_OUTPUT. Replaced with node dist/extract-release-notes.js (from the feat/git-utils PR)
  • add dry-run boolean input to release-pipeline, and thread it to version-bump, npm-publish, and release-notes

tweaks:

  • update the stale-branches action and pin to the latest commit SHA
  • update validate-jira action to checkout the branch first before running the action. the ${{ github.action_path}} variable is not valid for shared workflows, only composite actions

@adrmachado-public
adrmachado-public marked this pull request as ready for review June 8, 2026 15:48
@adrmachado-public
adrmachado-public requested a review from a team June 8, 2026 15:48

@GoogilyBoogily GoogilyBoogily 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.

All comments around optimizing/simplifying what we have.

Comment thread .github/workflows/release-dry-run.yml Outdated
Comment thread .github/workflows/validate-scripts.yml Outdated
Comment thread actions/npm-publish/action.yml Outdated
Comment thread actions/npm-publish/action.yml Outdated
Comment thread actions/release-notes/action.yml Outdated
Comment thread actions/version-bump/action.yml Outdated

@GoogilyBoogily GoogilyBoogily 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.

👍 🐸

@codentacos codentacos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍 🧙

@adrmachado-public
adrmachado-public merged commit 7a01294 into main Jun 8, 2026
1 check passed
@adrmachado-public
adrmachado-public deleted the feat/dry-run-typescript branch June 8, 2026 17: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.

3 participants