Skip to content

Conversation

@remyduthu
Copy link
Contributor

The goal of this change is to remove the documentation related to this
variable as we've removed it.

References: MRGFY-6218

The goal of this change is to remove the documentation related to this
variable as we've removed it.

References: MRGFY-6218
Change-Id: Ieffb84dc6acc5cf9c3d28940fde4ac3e863d58e8
@mergify mergify bot deployed to Mergify Merge Protections December 10, 2025 10:26 Active
@remyduthu remyduthu self-assigned this Dec 10, 2025
@remyduthu remyduthu marked this pull request as ready for review December 10, 2025 10:26
Copilot AI review requested due to automatic review settings December 10, 2025 10:26
@mergify
Copy link
Contributor

mergify bot commented Dec 10, 2025

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,})

Copy link

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

This PR removes all documentation references to the MERGIFY_TEST_FLAKY_DETECTION environment variable, which has been removed from the product. The change ensures documentation stays in sync with the current implementation.

  • Removed MERGIFY_TEST_FLAKY_DETECTION from code examples in Jenkins and GitHub Actions setup guides
  • Removed documentation of this variable from the "Key Environment Variables" sections
  • Minor alphabetization of imports in the Jenkins documentation file

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/content/docs/ci-insights/setup/jenkins.mdx Removed MERGIFY_TEST_FLAKY_DETECTION from two code examples and the environment variables list; alphabetized imports
src/content/docs/ci-insights/setup/github-actions.mdx Removed MERGIFY_TEST_FLAKY_DETECTION from code example and the environment variables list

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

mergify bot added a commit that referenced this pull request Dec 15, 2025
@mergify
Copy link
Contributor

mergify bot commented Dec 15, 2025

Merge Queue Status

✅ The pull request has been merged at 1922fec

This pull request spent 1 minute 47 seconds in the queue, including 1 minute 30 seconds running CI.
The checks were run on draft #9666.

Required conditions to merge

@mergify mergify bot added the queued label Dec 15, 2025
@mergify mergify bot merged commit 36b220a into main Dec 15, 2025
14 checks passed
@mergify mergify bot deleted the devs/remyduthu/remove-flaky-detection-variable/Ieffb84dc6acc5cf9c3d28940fde4ac3e863d58e8 branch December 15, 2025 10:18
@mergify mergify bot removed the queued label Dec 15, 2025
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.

4 participants