Skip to content

Conversation

@dblock
Copy link
Member

@dblock dblock commented Dec 30, 2025

Summary

  • Migrated from custom Danger setup to danger-pr-comment workflow framework following slack-ruby/slack-ruby-client#581 and slack-ruby/slack-ruby-bot-server#181
  • Replaced manual Danger workflow with reusable workflows from numbata/danger-pr-comment
  • Removed ruby-grape-danger dependency in favor of standard Danger plugins
  • Added new Danger gems: danger, danger-pr-comment, danger-changelog, and danger-toc
  • Updated Dangerfile to import danger-pr-comment and added changelog.check! and toc.check!

Changes

  • Updated .github/workflows/danger.yml to use danger-pr-comment reusable workflow
  • Added .github/workflows/danger-comment.yml for posting PR comments
  • Updated Gemfile to include danger, danger-pr-comment, danger-changelog, and danger-toc
  • Removed ruby-grape-danger dependency from Gemfile
  • Updated Dangerfile to import danger-pr-comment and added changelog.check! and toc.check!
  • Removed ruby-grape-danger import from Dangerfile

Test plan

  • Verify Danger workflow runs successfully on this PR
  • Verify Danger comment is posted to this PR
  • Verify changelog check works
  • Verify TOC check works

🤖 Generated with Claude Code

@dblock dblock force-pushed the migrate-danger-pr-comment branch from 72b8cf0 to 18d23ef Compare December 30, 2025 19:24
- Migrated from custom Danger setup to danger-pr-comment workflow framework
- Replaced manual Danger workflow with reusable workflows from numbata/danger-pr-comment
- Added .github/workflows/danger.yml using danger-pr-comment reusable workflow
- Added .github/workflows/danger-comment.yml for posting PR comments
- Updated Gemfile to include danger, danger-pr-comment, danger-changelog, and danger-toc
- Updated Dangerfile to import danger-pr-comment and added changelog.check! and toc.check!

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
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