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.

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

feat(platform/azure): Recreate Auto-Approve on PR-Update #22028

Merged
merged 11 commits into from May 11, 2023
Merged

feat(platform/azure): Recreate Auto-Approve on PR-Update #22028

merged 11 commits into from May 11, 2023

Conversation

horihel
Copy link
Contributor

@horihel horihel commented May 8, 2023

Changes

This change will allow renovate to re-approve a PR on azure if autoApprove is enabled. This should solve #15664

This makes autoApprove behavior more intuitive, as updated PRs now have the same constraints as fresh PRs.

Context

See description and discussion in the issue #15664 .

Code (and test) is mostly copied/modified from the corresponding snippet in createPR().

As these are my very first lines of typescript, the code (and especially the test) might be lacking in quality. If that is the case, I'm thankful for pointers.

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

Signed-off-by: Heiko Helmle <heiko.helmle@horiba.com>
Signed-off-by: Heiko Helmle <heiko.helmle@horiba.com>
Signed-off-by: Heiko Helmle <heiko.helmle@horiba.com>
@horihel horihel requested a review from JamieMagee as a code owner May 8, 2023 11:48
Signed-off-by: Heiko Helmle <heiko.helmle@horiba.com>
Signed-off-by: Heiko Helmle <heiko.helmle@horiba.com>
@rarkins rarkins requested a review from viceice May 9, 2023 06:40
Signed-off-by: Heiko Helmle <heiko.helmle@horiba.com>
@JamieMagee JamieMagee enabled auto-merge May 10, 2023 16:20
@JamieMagee JamieMagee added this pull request to the merge queue May 11, 2023
Merged via the queue into renovatebot:main with commit 08233ff May 11, 2023
11 checks passed
@renovate-release
Copy link
Collaborator

🎉 This PR is included in version 35.79.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

mjunker pushed a commit to mjunker/renovate that referenced this pull request May 17, 2023
…#22028)

Signed-off-by: Heiko Helmle <heiko.helmle@horiba.com>
Co-authored-by: Jamie Magee <jamie.magee@gmail.com>
Co-authored-by: Rhys Arkins <rhys@arkins.net>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants