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

CD enable next-build-number #4168

Merged

Conversation

akomakom
Copy link
Contributor

Link to GitHub repository

https://github.com/jenkinsci/next-build-number-plugin

When modifying release permission

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

  • @akomakom

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:

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

jenkinsci/next-build-number-plugin#25

Tasks

No tasks being tracked yet.

Reviewer checklist

There are IRC Bot commands for it.

@akomakom akomakom requested a review from a team as a code owner November 12, 2024 16:58
@timja timja merged commit 2db75b8 into jenkins-infra:master Nov 12, 2024
3 checks passed
@akomakom
Copy link
Contributor Author

@timja if I'm reading the auto CD setup instructions right, after this got merged I should be able to access tokens for my plugin at https://github.com/jenkinsci/next-build-number-plugin/settings/secrets/actions ...

Did I miss a step?
https://www.jenkins.io/doc/developer/publishing/releasing-cd/#enable-cd-permissions

Thanks!

@timja
Copy link
Member

timja commented Nov 13, 2024

They are there:
image

@akomakom
Copy link
Contributor Author

@timja looks like I don't have appropriate privileges to the repo. I thought perhaps that this process would fix that.

@timja
Copy link
Member

timja commented Nov 13, 2024

Fixed

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