Skip to content

Migrate Danger to danger-pr-comment workflows - #50

Closed
dblock wants to merge 1 commit into
mongoid:masterfrom
dblock:dblock/danger-pr-comment
Closed

dblock wants to merge 1 commit into
mongoid:masterfrom
dblock:dblock/danger-pr-comment

Conversation

@dblock

@dblock dblock commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Follow mongoid/mongoid-slug#277: import danger-pr-comment, validate changelog entries, and separate read-only PR checks from report-based comment publishing through the pinned reusable workflow.

Add an isolated Gemfile.danger with danger, danger-changelog 0.9.0, and danger-pr-comment. This allows modern Ruby 3.4 to run the PR checks without installing the legacy application dependency stack. Remove mongoid-danger from the application Gemfile and the redundant Travis Danger invocation; preserve the application test matrix.

Validation: isolated bundle installs, changed Ruby files pass syntax checks, GitHub/Travis YAML parses, and git diff --check passes. Application tests were not run for this tooling-only change.

Prepared by AI on behalf of @dblock.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock

dblock commented Oct 17, 2026

Copy link
Copy Markdown
Contributor Author

AI posting on behalf of @dblock: incorporated into #51 alongside the CI modernization, including the isolated Danger bundle and fork-safe report/comment split. Closing this duplicate in favor of #51.

@dblock dblock closed this Oct 17, 2026
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