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

Renovate config to automate @edx namespaced minor/patch version upgrades #770

Conversation

abdullahwaheed
Copy link
Contributor

@abdullahwaheed abdullahwaheed commented Aug 23, 2022

Ticket

Create common renovate config that automates @edx namespaced minor/patch version upgrades

What has changed

Added Renovate config to update and auto merge @edx name spaced minor and patch packages.
Added auto merge schedule to at any time as overall schedule is for weekend only

@mphilbrick211
Copy link

mphilbrick211 commented Oct 26, 2022

Hi @abdullahwaheed! If you plan to proceed with this PR, you will need to rebase as the branch is out-of-date. Thank you!

@mphilbrick211 mphilbrick211 added the waiting on author PR author needs to resolve review requests, answer questions, fix tests, etc. label Nov 7, 2022
@arbrandes arbrandes force-pushed the abdullahwaheed/renovate-bot-edx-packages-auto-update branch from 6ce584a to 66ad192 Compare November 30, 2022 13:26
"matchPackagePatterns": ["@edx"],
"matchUpdateTypes": ["minor", "patch"],
"automerge": true,
"automergeSchedule": ["at any time"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it okay to schedule it to anytime? (please enlighten me since I may not have full context on this.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this will merge the PR of @edx minor and patch package upgrades anytime and automatically. Mostly the minor and patch upgrades doesn't have breaking changes so we can safely merge it anytime.

@mphilbrick211 mphilbrick211 removed the waiting on author PR author needs to resolve review requests, answer questions, fix tests, etc. label Dec 7, 2022
@jmbowman jmbowman added the waiting on author PR author needs to resolve review requests, answer questions, fix tests, etc. label Dec 9, 2022
@jmbowman
Copy link

jmbowman commented Dec 9, 2022

Added back the label to flag @abdullahwaheed that this seems to be waiting for his action (just rebase and merge, as far as I can tell).

@abdullahwaheed
Copy link
Contributor Author

done rebasing, ready to merge

@mphilbrick211 mphilbrick211 removed the waiting on author PR author needs to resolve review requests, answer questions, fix tests, etc. label Dec 12, 2022
@mphilbrick211
Copy link

@ansabgillani - this looks ready for you review/merge. Thanks!

@mphilbrick211
Copy link

@ansabgillani - this looks ready for you review/merge. Thanks!

Actually, sorry @ansabgillani - looks like @Ali-D-Akbar needs to complete review first.

@Ali-D-Akbar would you mind taking a look at this?

@Ali-D-Akbar Ali-D-Akbar merged commit a690eca into openedx:master Dec 13, 2022
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.

5 participants