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

Adopt rest-list-parameter-plugin #4044

Merged

Conversation

mymarche
Copy link
Contributor

@mymarche mymarche commented Aug 21, 2024

Link to GitHub repository

https://github.com/jenkinsci/rest-list-parameter-plugin

When modifying release permission

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

  • @h1dden_da3m0n
  • @mymarche

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

CD checklist (for submitters)

Reviewer checklist

There are IRC Bot commands for it.

@mymarche mymarche requested a review from a team as a code owner August 21, 2024 09:33
@mymarche
Copy link
Contributor Author

mymarche commented Aug 21, 2024

Hi, @h1dden-da3m0n , Could you please approve?

@alecharp
Copy link
Contributor

We are over the 2 weeks period. Sorry about that @mymarche. Are you still interest to become a maintainer of the plugin?

@mymarche
Copy link
Contributor Author

Hello, @alecharp! Yes, I'm still interested.

@timja timja merged commit 10b9522 into jenkins-infra:master Nov 19, 2024
3 checks passed
@timja
Copy link
Member

timja commented Nov 19, 2024

Hi, welcome aboard!

Please go to https://github.com/jenkinsci to accept the invite

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants