Skip to content

build: update pinned ci-tools to a98f5d0 - #306

Merged
marc0olo merged 1 commit into
mainfrom
build/ci-tools-a98f5d0
Sep 25, 2026
Merged

marc0olo merged 1 commit into
mainfrom
build/ci-tools-a98f5d0

Conversation

@marc0olo

Copy link
Copy Markdown
Member

Picks up dfinity/ci-tools#94. bdfe993..a98f5d0 is only that commit.

Since reuse_branch (dfinity/ci-tools#80), the changelog pull request is force pushed on every push to main, and those pushes came from the default token as github-actions[bot]. Its workflows then waited for approval, and required pull_request_target workflows such as the External PR Ruleset never ran, which blocks the pull request even when approved (e.g. #290). With #94, the branch is pushed with the app token instead.

🤖 Generated with Claude Code

Copilot AI lite review requested due to automatic review settings September 25, 2026 14:06
@marc0olo
marc0olo requested a review from a team as a code owner September 25, 2026 14:06

Copilot AI 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.

Copilot review overview

🟢 Approval recommended

No unresolved issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Updates pinned dfinity/ci-tools references to a98f5d0, including the changelog app-token fix.

Changes:

  • Refreshes CI, release, testing, and validation workflow pins.
  • Updates changelog and release automation workflows.
File Description
.github/​workflows/​release.yml Updates release and documentation pins
.github/​workflows/​lint.yml Updates PNPM setup pin
.github/​workflows/​generate-changelog.yml Updates changelog workflow
.github/​workflows/​e2e-test-nodejs.yml Updates E2E setup pins
.github/​workflows/​create-release-pr.yml Updates release PR tooling pins
.github/​workflows/​commitizen.yml Updates validation workflow pins

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@marc0olo
marc0olo added this pull request to the merge queue Sep 25, 2026
Merged via the queue into main with commit 5504af7 Sep 25, 2026
18 checks passed
@marc0olo
marc0olo deleted the build/ci-tools-a98f5d0 branch September 25, 2026 14:23
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