ci(renovate): keep ci.yaml mergify-cli pin in sync with the action default#10
Conversation
|
This pull request is part of a Mergify stack:
|
Merge Protections🟢 All 7 merge protections satisfied — ready to merge. Show 7 satisfied protections🟢 ⛓️ Depends-On RequirementsRequirement based on the presence of
🟢 🤖 Continuous Integration
🟢 👀 Review Requirements
🟢 Enforce conventional commitMake sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 Reviews
🟢 📕 PR description
🟢 🚦 Auto-queueWhen all merge protections are satisfied, this pull request will be queued automatically. |
…fault The pinned-install dogfooding test in ci.yaml hardcoded the mergify_cli_version, but no customManager covered the file, so every mergify-cli release bumped only action.yml + README.md and left ci.yaml stranded on the previous version (as commit 8d75e33 did). Refactor the pinned-install assertion to compare against an EXPECTED_MERGIFY_CLI_VERSION env literal instead of three inline version literals, then add a regex customManager (mirroring the action.yml/README.md ones) that bumps the with: input and the expected env in lockstep. Both matchStrings are anchored to their mergify-specific YAML key, tolerate an optional quote, and require a digit-led value, so they cannot hijack an unrelated quoted assertion or env and leave the neighbouring `latest` install test untouched. Change-Id: Ia4f0d81a819bb8fa2cc205a27d3e38d2197340c5 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
46c5501 to
ed45962
Compare
Revision history
|
Merge Queue Status
|
Merge Queue Status
This pull request spent 15 minutes 8 seconds in the queue, including 2 seconds running CI. Required conditions to merge
ReasonPull request #10 could not be retargeted onto the queue base branch HintThis pull request is stacked on another whose head branch still exists after merging, so GitHub never retargeted it onto the queue base branch. Re-push the stack (for example with Requeued — the merge queue status continues in this comment ↓. |
|
@Mergifyio queue |
Merge Queue Status
This pull request spent 15 minutes 30 seconds in the queue, including 8 seconds running CI. Required conditions to merge
ReasonPull request #10 could not be retargeted onto the queue base branch HintThis pull request is stacked on another whose head branch still exists after merging, so GitHub never retargeted it onto the queue base branch. Re-push the stack (for example with Requeued — the merge queue status continues in this comment ↓. |
|
@Mergifyio queue |
Merge Queue Status
This pull request spent 19 seconds in the queue, with no time running CI. ReasonThe pull request conflicts with the base branch HintYou should update or rebase your pull request. If you want to requeue this pull request, you can post a Requeued — the merge queue status continues in this comment ↓. |
ed45962 to
e3a334f
Compare
|
@sileht this pull request is now in conflict 😩 |
Merge Queue Status
This pull request spent 17 seconds in the queue, including 4 seconds running CI. Required conditions to merge
|
The pinned-install dogfooding test in ci.yaml hardcoded the
mergify_cli_version, but no customManager covered the file, so every
mergify-cli release bumped only action.yml + README.md and left ci.yaml
stranded on the previous version (as commit 8d75e33 did).
Refactor the pinned-install assertion to compare against an
EXPECTED_MERGIFY_CLI_VERSION env literal instead of three inline version
literals, then add a regex customManager (mirroring the
action.yml/README.md ones) that bumps the with: input and the expected
env in lockstep. Both matchStrings are anchored to their mergify-specific
YAML key, tolerate an optional quote, and require a digit-led value, so
they cannot hijack an unrelated quoted assertion or env and leave the
neighbouring
latestinstall test untouched.Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com
Depends-On: #9