-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add markewaite as maintainer of 3 plugins #4757
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
base: master
Are you sure you want to change the base?
Add markewaite as maintainer of 3 plugins #4757
Conversation
Pull request to be merged: * jenkinsci/text-finder-plugin#295 Current maintainers that can approve the addition: * @basil * @jglick
Pull request to merge: * jenkinsci/throttle-concurrent-builds-plugin#339 Current maintainers that can approve the addition: * @abayer * @basil * @oleg-nenashev
Pull request to be merged: * jenkinsci/timestamper-plugin#394 Current maintainers that can approve my addition: * @basil * @StevenGBrown
Pull request to be merged: * jenkinsci/ws-cleanup-plugin#267 Current maintainers that can approve my addition as a maintainer: * @basil * @vjuranek * @PaJaSoft
|
Approved |
|
I am not even sure what the |
You became a maintainer of that plugin in 2016 with commit a673619 , the initial upload of permissions. |
|
jenkinsci/text-finder-plugin#12 (comment) and seven years after that I certainly have no idea. Consider it abandoned, whatever it is. |
Pull request has been merged to ws-cleanup
Add markewaite as maintainer of 3 plugins
I've included these changes in a single pull request because they can all be approved by @basil. If he doesn't approve them, I'm willing to wait the 2 weeks for the adoption request to be approved.
Add markewaite as a timestamper plugin maintainer
Pull request to be merged:
Current maintainers that can approve my addition:
Add markewaite as a maintainer of throttle-concurrent-builds plugin
Pull request to merge:
Current maintainers that can approve the addition:
Add markewaite as a text-finder plugin maintainer
Pull request to be merged:
Current maintainers that can approve the addition:
Link to GitHub repository
When modifying release permission
List the GitHub usernames of the users who should have commit permissions below:
@MarkEWaiteThis is needed in order to cut releases of the plugin or component.
If you are modifying the release permission of your plugin or component, fill out the following checklist:
Release permission checklist (for submitters)
When enabling automated releases (cd: true)
Follow the documentation to ensure, your pull request is set up properly. Don't merge it yet.
In case of changes requested by the hosting team, an open PR facilitates future reviews, without derailing work across multiple PRs.
Link to the PR enabling CD in your plugin
CD checklist (for submitters)
Reviewer checklist
$pluginId Developersteam hasAdminpermissions while granting the access.@Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.There are IRC Bot commands for it.