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

Add permissions for drulli for Cobertura and JaCoCo #4184

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

uhafner
Copy link
Contributor

@uhafner uhafner commented Nov 19, 2024

Link to GitHub repository

When modifying release permission

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

  • @uhafner

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.

@uhafner uhafner requested a review from a team as a code owner November 19, 2024 16:05
@NotMyFault NotMyFault enabled auto-merge (squash) November 19, 2024 16:07
@NotMyFault NotMyFault merged commit 5a48d6a into jenkins-infra:master Nov 19, 2024
3 checks passed
@Wadeck
Copy link
Collaborator

Wadeck commented Nov 19, 2024

@NotMyFault was the process related to adoption changed? There are 6+2 maintainers and the merge was done in 3 minutes, I am a bit surprised.
Jacoco was marked up for adoption, so I can understand, but Cobertura not.

(it's totally possible that I missed some recent changes ;) )

@NotMyFault
Copy link
Member

The cobertura plugin has had no activity in commits and PR reviews in over a year, while having 6 maintainers.
I consider this as abandoned, making the plugin eligible for adoption.

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.

3 participants