Skip to content

docs: rewrite RSpec page for rspec-mergify gem integration#10813

Merged
mergify[bot] merged 1 commit intomainfrom
devs/sileht/rspec/I35f8ea6dee585b97c541796c4776406c500a4399
Apr 3, 2026
Merged

docs: rewrite RSpec page for rspec-mergify gem integration#10813
mergify[bot] merged 1 commit intomainfrom
devs/sileht/rspec/I35f8ea6dee585b97c541796c4776406c500a4399

Conversation

@sileht
Copy link
Copy Markdown
Member

@sileht sileht commented Apr 2, 2026

Replace the JUnit XML upload approach with rspec-mergify as the primary
integration method. The gem automatically uploads test results to CI
Insights when MERGIFY_TOKEN is set, matching the pytest-mergify pattern.

The previous JUnit XML approach is preserved as an "Alternative" section.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

Copilot AI review requested due to automatic review settings April 2, 2026 14:17
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 2, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 👀 Review Requirements

Wonderful, this rule succeeded.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

@mergify mergify bot had a problem deploying to Mergify Merge Protections April 2, 2026 14:17 Failure
@mergify mergify bot requested a review from a team April 2, 2026 14:19
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the CI Insights documentation for RSpec to recommend the rspec-mergify gem as the primary integration path (similar to the existing pytest-mergify docs), while keeping the previous JUnit XML + GitHub Action upload flow as an alternative.

Changes:

  • Rewrites the RSpec CI Insights page to focus on rspec-mergify installation and minimal workflow setup (export MERGIFY_TOKEN).
  • Adds an “Environment Variables” section describing configuration options for the gem.
  • Preserves the legacy approach as an “Alternative: Manual JUnit XML Upload” section with inline YAML examples.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sileht sileht marked this pull request as ready for review April 2, 2026 14:58
kozlek
kozlek previously approved these changes Apr 2, 2026
@mergify mergify bot requested a review from a team April 2, 2026 15:45
JulianMaurin
JulianMaurin previously approved these changes Apr 2, 2026
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 2, 2026

Merge Queue Status

This pull request spent 14 hours 12 minutes 55 seconds in the queue, including 2 minutes 7 seconds running CI.

Required conditions to merge

Reason

Pull request #10813 has been dequeued

The pull request rule doesn't match anymore

Hint

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.
If you do update this pull request, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio queue comment.

mergify bot added a commit that referenced this pull request Apr 2, 2026
@mergify mergify bot added the queued label Apr 2, 2026
mergify bot added a commit that referenced this pull request Apr 3, 2026
mergify bot added a commit that referenced this pull request Apr 3, 2026
mergify bot added a commit that referenced this pull request Apr 3, 2026
jd
jd previously requested changes Apr 3, 2026
Replace the JUnit XML upload approach with rspec-mergify as the primary
integration method. The gem automatically uploads test results to CI
Insights when MERGIFY_TOKEN is set, matching the pytest-mergify pattern.

The previous JUnit XML approach is preserved as an "Alternative" section.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Change-Id: I35f8ea6dee585b97c541796c4776406c500a4399
@sileht sileht force-pushed the devs/sileht/rspec/I35f8ea6dee585b97c541796c4776406c500a4399 branch from d01aaf1 to dfcbfac Compare April 3, 2026 06:58
@mergify mergify bot dismissed kozlek’s stale review April 3, 2026 06:59

Pull request has been modified.

@mergify mergify bot dismissed stale reviews from jd and JulianMaurin April 3, 2026 06:59

Pull request has been modified.

@mergify mergify bot removed the dequeued label Apr 3, 2026
@mergify mergify bot deployed to Mergify Merge Protections April 3, 2026 06:59 Active
@mergify mergify bot requested a review from a team April 3, 2026 07:01
@mergify mergify bot requested a review from a team April 3, 2026 08:01
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 3, 2026

Merge Queue Status

This pull request spent 2 minutes 25 seconds in the queue, including 2 minutes 11 seconds running CI.

Required conditions to merge

mergify bot added a commit that referenced this pull request Apr 3, 2026
@mergify mergify bot added the queued label Apr 3, 2026
@mergify mergify bot merged commit 4b69296 into main Apr 3, 2026
9 checks passed
@mergify mergify bot deleted the devs/sileht/rspec/I35f8ea6dee585b97c541796c4776406c500a4399 branch April 3, 2026 08:07
@mergify mergify bot removed the queued label Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants