Skip to content

Remove useBeta property from pom.xml#2627

Merged
timja merged 5 commits intojenkinsci:masterfrom
strangelookingnerd:remove_use_beta
May 6, 2025
Merged

Remove useBeta property from pom.xml#2627
timja merged 5 commits intojenkinsci:masterfrom
strangelookingnerd:remove_use_beta

Conversation

@strangelookingnerd
Copy link
Copy Markdown
Contributor

Now that the permission Jenkins.MANAGE is out of beta (jenkinsci/jenkins#10183) the useBeta property is no longer required.

This is still a draft, since it will take some time until that change is shipped in an LTS and this plugin updates to it.
I would like to use this PR as a reminder and update / merge it once the requirements are met.

Testing done

Validated with mvn clean verify

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@timja
Copy link
Copy Markdown
Member

timja commented Feb 20, 2025

This is still a draft, since it will take some time until that change is shipped in an LTS and this plugin updates to it.
I would like to use this PR as a reminder and update / merge it once the requirements are met.

No problem to update to a weekly if needed, don't need to wait for LTS, but this isn't that important so it can wait...

@timja timja added the developer A PR that merits a release for plugin developers label May 6, 2025
@strangelookingnerd
Copy link
Copy Markdown
Contributor Author

The required change in the Jenkins core (jenkinsci/jenkins#10183) was shipped with the latest LTS 2.504.1.
While there is no urgency to merge this PR, it should not get burried either 👍🏼

@strangelookingnerd strangelookingnerd marked this pull request as ready for review May 6, 2025 13:44
@strangelookingnerd strangelookingnerd requested a review from a team as a code owner May 6, 2025 13:44
@timja timja enabled auto-merge (squash) May 6, 2025 13:47
@timja timja merged commit ba0fb68 into jenkinsci:master May 6, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

developer A PR that merits a release for plugin developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants