Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Reduce usage of Ant #10059

Merged
merged 1 commit into from
Dec 15, 2024
Merged

Reduce usage of Ant #10059

merged 1 commit into from
Dec 15, 2024

Conversation

basil
Copy link
Member

@basil basil commented Dec 13, 2024

While reading this code recently, I noticed we were using Ant for functionality that is now present in the Java Platform. Easier to maintain code that has fewer third-party library dependencies.

Testing done

mvn clean verify -Dtest=hudson.model.DirectoryBrowserSupportTest,jenkins.util.VirtualFileTest

Proposed changelog entries

N/A

Proposed upgrade guidelines

N/A

Submitter checklist

Preview Give feedback

Desired reviewers

@mention

Before the changes are marked as ready-for-merge:

Maintainer checklist

Preview Give feedback

@basil basil added the skip-changelog Should not be shown in the changelog label Dec 13, 2024
@MarkEWaite
Copy link
Contributor

/
This PR is now ready for merge. We will merge it after approximately 24 hours if there is no negative feedback.

/label ready-for-merge

@comment-ops-bot comment-ops-bot bot added the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label Dec 14, 2024
@timja timja merged commit e4bff05 into jenkinsci:master Dec 15, 2024
15 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback skip-changelog Should not be shown in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants