Skip to content

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

Merged
dblock merged 3 commits intocodegram:masterfrom
dblock:danger-pr-comment
Feb 3, 2026
Merged

Migrate Danger to use danger-pr-comment workflow#308
dblock merged 3 commits intocodegram:masterfrom
dblock:danger-pr-comment

Conversation

@dblock
Copy link
Collaborator

@dblock dblock commented Feb 3, 2026

Summary

  • Replaced inline Danger workflow with reusable workflows from numbata/danger-pr-comment.
  • Added danger-comment.yml workflow to post Danger results as PR comments via workflow_run, enabling fork PR support.
  • Added danger and danger-pr-comment gems to Gemfile.

Test plan

  • Verify Danger workflow triggers on a PR.
  • Verify Danger Comment workflow posts results after Danger completes.

🤖 Generated with Claude Code

dblock and others added 3 commits February 3, 2026 08:48
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Fixes undefined method 'github' in dry_run mode used by danger-pr-comment workflow.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove `require 'minitest/unit'` which was removed in recent minitest
versions, and use `Minitest::Assertions` instead of `MiniTest::Assertions`.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coveralls
Copy link

Coverage Status

coverage: 94.7%. remained the same
when pulling dc21e63 on dblock:danger-pr-comment
into 2b34b4b on codegram:master.

@dblock dblock merged commit 7de1f25 into codegram:master Feb 3, 2026
9 checks passed
@dblock dblock deleted the danger-pr-comment branch February 3, 2026 14:00
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.

2 participants