Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't update renovate PRs via mergify #988

Merged
merged 2 commits into from
Dec 16, 2024
Merged

Don't update renovate PRs via mergify #988

merged 2 commits into from
Dec 16, 2024

Conversation

tnotheis
Copy link
Member

Readiness checklist

  • I added/updated unit tests.
  • I added/updated integration tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.

@tnotheis tnotheis added the ci Continuous Integration related stuff label Dec 16, 2024
@tnotheis tnotheis self-assigned this Dec 16, 2024
Copilot
Copilot AI reviewed Dec 16, 2024
View reviewed changes

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.
jkoenig134
jkoenig134 approved these changes Dec 16, 2024
View reviewed changes
Copy link
Member

@jkoenig134 jkoenig134 left a comment

Choose a reason for hiding this comment

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

@tnotheis
Copy link
Member Author

Because renovate doesn't like anyone messing with its PRs. ;) See for example #985 (comment)

I assume it does the update own its own.

@tnotheis tnotheis merged commit 9b49f85 into main Dec 16, 2024
27 checks passed
@tnotheis tnotheis deleted the dont-update-renovate-prs-via-mergify branch December 16, 2024 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@jkoenig134 jkoenig134 jkoenig134 approved these changes

Assignees

@tnotheis tnotheis

Labels
ci Continuous Integration related stuff
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tnotheis @jkoenig134