-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Update CloudBees team members #4197
Update CloudBees team members #4197
Conversation
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.
LGTM
Co-authored-by: Baptiste Mathus <[email protected]>
@alecharp @timja CI failed because @PereBueno wasn't logged in. Now he's logged in, can you retry the CI please? |
trying to retrigger the CI |
failing again. Might it be because the gitthub and Jira users are different? PereBueno vs pbuenoyerbes |
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.
👍 thanks Fran :)
it's the LDAP username. See the release permission checklist. |
@alecharp can you please merge the PR? I don't have any auto-merge button |
Link to GitHub repository
No repo
When modifying release permission
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
There are IRC Bot commands for it.