Skip to content

Migrate Danger to use danger-pr-comment workflow - #49

Merged
dblock merged 1 commit into
masterfrom
dblock/migrate-danger-pr-comment
Aug 21, 2026
Merged

Migrate Danger to use danger-pr-comment workflow#49
dblock merged 1 commit into
masterfrom
dblock/migrate-danger-pr-comment

Conversation

@dblock

@dblock dblock commented Aug 21, 2026

Copy link
Copy Markdown
Owner

Migrate Danger to use the danger-pr-comment reusable workflow, replacing the hardcoded/expired personal access token approach, similar to dblock/ruby-link-checker#14 and dblock/ruby-link-checker#16.

  • Replace inline bundle exec danger step in danger.yml with a call to numbata/danger-pr-comment/.github/workflows/danger-run.yml@v0.1.0 (uses GITHUB_TOKEN, no more hardcoded PAT).
  • Add danger-comment.yml workflow (triggered on Danger workflow completion) to post the PR comment, with actions: read, issues: write, pull-requests: write permissions.
  • Add danger-pr-comment gem and import its Dangerfile plugin.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock force-pushed the dblock/migrate-danger-pr-comment branch from bf96372 to 6580f6e Compare August 21, 2026 17:56
@dblock
dblock merged commit e22564b into master Aug 21, 2026
15 checks passed
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