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

plugin-openid: Enable CD Permissions #3793

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

panicking
Copy link
Contributor

Prepare plugin delivery using jenkins CI. Follow the documentation found at:

https://www.jenkins.io/doc/developer/publishing/releasing-cd/

Link to GitHub repository

https://github.com/jenkinsci/openid-plugin

When modifying release permission

List the GitHub usernames of the users who should have commit permissions below:

  • panicking

This 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:

  • The usernames of the users added to the "developers" section in the .yml file are the same the users use to log in to accounts.jenkins.io. panicking
    team to approve this request.

Link to the PR enabling CD in your plugin

jenkinsci/openid-plugin@a02cb13

Prepare plugin delivery using jenkins CI. Follow the documentation
found at:

https://www.jenkins.io/doc/developer/publishing/releasing-cd/

Signed-off-by: Michael Trimarchi <[email protected]>
@panicking panicking requested a review from a team as a code owner March 1, 2024 18:05
@NotMyFault
Copy link
Member

Link to the PR enabling CD in your plugin

jenkinsci/openid-plugin@a02cb13

is incomplete and actually unrelated to CD at all. Please take a look at the documentation how to incrementalify your plugin and make the needed modifications as a PR we can review.

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.

2 participants