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

Remove transformPom #520

Merged
merged 1 commit into from
Apr 11, 2023
Merged

Remove transformPom #520

merged 1 commit into from
Apr 11, 2023

Conversation

basil
Copy link
Member

@basil basil commented Apr 11, 2023

I could not find any usages of this in open-source or proprietary code, so let us remove this dead code. The use case demonstrated in the unit test did not make sense to me, anyway: why would we want to upgrade a plugin parent POM to a core version? This code seems better left behind in the rear-view mirror.

@basil basil added the removed label Apr 11, 2023
@basil basil requested a review from jtnord April 11, 2023 05:16
@basil basil requested a review from a team as a code owner April 11, 2023 05:16
@basil basil merged commit 2e05ec6 into jenkinsci:master Apr 11, 2023
@basil basil deleted the transformPom branch April 11, 2023 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants