-
Notifications
You must be signed in to change notification settings - Fork 203
feat: delete stale branches script #11395
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 occa 8000 sionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
be053c0
to
60f1a7e
Compare
Sandbox results
|
2c0973a
to
e060efa
Compare
ocis CI output
|
4aeaf9c
to
63b2f7d
Compare
"name": "delete stale branches", | ||
"image": OC_CI_ALPINE, | ||
"commands": [ | ||
"./scripts/delete-stale-branches.sh", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The goal is to run this script in nighly pipeline.
With this command it will be run as DRY_RUN by default.
I suggest to merge as DRY_RUN and confirm the branch filtering before disabling DRY_RUN:
Branch Filtering:
IGNORE_BRANCHES_REGEX: ^(main$|master$|stable-|release-|docs$|docs-stable-)$
Ignored branches:$
docs
docs-stable-2.0
docs-stable-3.0
docs-stable-4.0
docs-stable-5.0
docs-stable-7.0
docs-stable-7.1
master
release-7.1.0
release-7.1.0-rc.1
release-7.1.1
stable-2.0
stable-3.0
stable-4.0
stable-5.0
stable-6.5
stable-7.0
stable-7.0-new
stable-7.1
63b2f7d
to
48a769b
Compare
|
@@ -427,7 +427,8 @@ def main(ctx): | |||
build_release_helpers = \ | |||
changelog() + \ | |||
docs() + \ | |||
licenseCheck(ctx) | |||
licenseCheck(ctx) + \ | |||
deleteStaleBranches(ctx) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this correct pipeline to add to?
], | ||
"trigger": { | ||
"ref": [ | ||
"refs/heads/master", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like nighly trigger on master. Won't run on each PR commit
@mklos-kw this is a VERY COOL script! |
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropria 8000 te):
Types of changes
Checklist: