Skip to content

ci(mergify): upgrade configuration to current format#26

Open
mergify[bot] wants to merge 1 commit intomainfrom
mergify/configuration-deprecated-update
Open

ci(mergify): upgrade configuration to current format#26
mergify[bot] wants to merge 1 commit intomainfrom
mergify/configuration-deprecated-update

Conversation

@mergify
Copy link

@mergify mergify bot commented Nov 22, 2024

Hey there! 👋

We've noticed that your Mergify configuration is using some deprecated fields.

No worries—we've got your back! This automated PR updates your config to align with the newest standards, ensuring everything keeps running smoothly.

Do not procrastinate! You must upgrade your configuration before 2025-01-31, otherwise your configuration will stop working.

What's Changed?

  • pull_request_rules → actions → queue → merge_method: To ease the configuration of queues, we are moving this attribute in the queue_rules definition. (deadline: 2025-01-31)

Why This Matters

Keeping your configuration up-to-date means you'll benefit from the latest features and improvements Mergify has to offer. Plus, it helps prevent any unexpected hiccups down the road.

Got Questions? We've Got Answers! 🙌

Is this update safe to merge?

Absolutely! We've made sure the changes are compatible with your current setup. Your workflows should continue to work just as before—if not better!

Do I need to do anything special after merging?

Nope! Just merge this PR, and you're all set. If you have any custom configurations, it's a good idea to give them a quick look to ensure everything's in order.

What if I run into issues or have concerns?

We're here to help! Feel free to reach out to our support team anytime.

Thanks for being awesome and keeping your configuration up-to-date! If you have any thoughts or need a hand, don't hesitate to let us know.

Happy merging! 🎉

@mergify mergify bot force-pushed the mergify/configuration-deprecated-update branch from d0135ad to c906292 Compare November 25, 2024 08:37
@QodoAI-Agent
Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Indentation Error

The new queue_conditions and merge_conditions sections appear misaligned, which can break YAML parsing. Ensure list items are indented exactly two spaces under their parent key.

queue_conditions:
  - base=main
  - status-success=nventive.UnoApplicationTemplate
  - "#approved-reviews-by>=2"
Missing Queue Parameters

The added queue: action in the second pull_request_rules entry lacks required method (and optionally name) fields, so it won't function as intended.

conditions: []
actions:
  queue:
Empty Conditions

The new pull request rule has an empty conditions: [], meaning it will never match any pull requests. Populate or remove this rule to avoid confusion.

conditions: []

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