feat(ci): add --config option and MERGIFY_CONFIG_PATH envvar to git-refs and scopes#1000
Conversation
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 🤖 Continuous IntegrationWonderful, this rule succeeded.
🟢 👀 Review RequirementsWonderful, this rule succeeded.
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 ReviewsWonderful, this rule succeeded.
🟢 📕 PR descriptionWonderful, this rule succeeded.
|
🧪 CI InsightsHere's what we observed from your CI run for 6fba92a. 🟢 All jobs passed!But CI Insights is watching 👀 |
There was a problem hiding this comment.
Pull request overview
Adds support for selecting the Mergify YAML config via CLI/environment for CI scope detection, with a regression test around MERGIFY_CONFIG_PATH behavior.
Changes:
- Adds
MERGIFY_CONFIG_PATHas an envvar source for theci scopes --configoption and tightens path validation. - Adds a test asserting that an empty
MERGIFY_CONFIG_PATHshould fall back to config auto-detection.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
mergify_cli/ci/cli.py |
Updates the scopes command --config Click option to read from MERGIFY_CONFIG_PATH and require an existing file. |
mergify_cli/tests/ci/test_cli.py |
Adds a test for “empty env var => autodetect config” behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Change-Id: I0b7bb94a7e7aff249ee90b650392dbae68f370fe
084364b to
6fba92a
Compare
Pull request has been modified.
Merge Queue StatusRule:
This pull request spent 7 seconds in the queue, including 1 second running CI. Required conditions to merge
|
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com