Skip to content

mergify: fix mergify config #823

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 25, 2025
Merged

Conversation

Nikhil-Ladha
Copy link
Contributor

fix deprecated mergify config

fix deprecated mergify config

Signed-off-by: Nikhil-Ladha <nikhilladha1999@gmail.com>
@Nikhil-Ladha Nikhil-Ladha requested a review from a team June 25, 2025 04:58
@Nikhil-Ladha
Copy link
Contributor Author

The mergify check fails for current config, but passes for the new config.
So, we should be good here.
Though, I am not sure if we need to manually merge the PR or so 🤔

@iPraveenParihar
Copy link
Member

@Nikhil-Ladha do we have link to this config deprecation?

Copy link
Member

@iPraveenParihar iPraveenParihar left a comment

Choose a reason for hiding this comment

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

The config LGTM.

@iPraveenParihar iPraveenParihar dismissed their stale review June 25, 2025 10:15

Actually, I noticed there is a PR from Mergify to update the config - #712

Can you compare your PR with it?

@Nikhil-Ladha
Copy link
Contributor Author

@Nikhil-Ladha do we have link to this config deprecation?

I couldn't find a link in the release docs or so, but the CI is failing with the details, as mentioned in my previous comment.

@Nikhil-Ladha
Copy link
Contributor Author

Actually, I noticed there is a PR from Mergify to update the config - #712

Can you compare your PR with it?

It has a similar change for the 2 fields which this PR adds, so we are good wrt that.

@iPraveenParihar
Copy link
Member

I think will need to merge it manually. @Rakshith-R PTAL

@nixpanic
Copy link
Collaborator

@Mergifyio queue

Not sure this works...

@nixpanic
Copy link
Collaborator

At least the new configuration is valid:
image

@nixpanic nixpanic merged commit 17a36e4 into csi-addons:main Jun 25, 2025
14 checks passed
Copy link

mergify bot commented Jun 25, 2025

queue

🟠 Waiting for conditions to match

  • -closed [📌 queue requirement]
  • -conflict [📌 queue requirement]
  • -draft [📌 queue requirement]
  • any of: [📌 queue -> configuration change requirements]
    • -mergify-configuration-changed
    • check-success = Configuration changed
  • any of: [🔀 queue conditions]
    • all of: [📌 queue conditions of queue default]
      • #approved-reviews-by >= 2 [🛡 GitHub branch protection]
      • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]

@Nikhil-Ladha Nikhil-Ladha deleted the update-mergify branch June 26, 2025 06:35
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.

3 participants