Skip to content

Remove dependency on org.jetbrains:annotations through test-harness#2683

Merged
timja merged 1 commit intojenkinsci:masterfrom
Vlatombe:remove-jetbrains-annotations
May 27, 2025
Merged

Remove dependency on org.jetbrains:annotations through test-harness#2683
timja merged 1 commit intojenkinsci:masterfrom
Vlatombe:remove-jetbrains-annotations

Conversation

@Vlatombe
Copy link
Copy Markdown
Member

It has been removed from the Jenkins plugins parent pom a while ago, and shouldn't be transitive.

Also filed vsch/flexmark-java#655 but since we don't depend on the latest and this library hasn't had a release for a while, it is easier to exclude it here.

Your checklist for this pull request

🚨 Please review the guidelines for contributing to this repository.

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side) and not your master branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or in Jenkins JIRA
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Did you provide a test-case? That demonstrates feature works or fixes the issue.

It has been removed from the Jenkins plugins parent pom a while ago, and
shouldn't be transitive.
@Vlatombe Vlatombe requested a review from a team as a code owner May 23, 2025 07:43
Vlatombe added a commit to Vlatombe/kubernetes-plugin that referenced this pull request May 23, 2025
Was removed from main through parent pom, but was still present in tests through casc test-harness.

Excluding until we pick up jenkinsci/configuration-as-code-plugin#2683
@timja timja added the developer A PR that merits a release for plugin developers label May 27, 2025
@timja timja merged commit f928046 into jenkinsci:master May 27, 2025
17 checks passed
@Vlatombe Vlatombe deleted the remove-jetbrains-annotations branch May 27, 2025 06:54
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.

3 participants