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 out of date jar and scripts #483

Merged
merged 2 commits into from
Feb 21, 2024
Merged

Remove out of date jar and scripts #483

merged 2 commits into from
Feb 21, 2024

Conversation

beatbrot
Copy link
Contributor

Using the maven wrapper is currently broken since the org.apache.maven.plugins:maven-enforcer-plugin plugin requires at least maven 3.6.3. The committed maven wrapper uses version 3.5.0.

This PR updates the wrapper straight to 3.9.6 which fixes the issue.

@garydgregory
Copy link
Member

We have jar files in the repository? Gross. I'd rather get rid of this since it will automatically go out of date with the next release, and the next one. I never use it. The GitHub CI doesn't either.

@beatbrot
Copy link
Contributor Author

I mean...personally, I like the wrapper scripts. But if you want, I can change this PR to simply remove the wrapper :) Should I do that?

@garydgregory
Copy link
Member

Hi @beatbrot
Let's see if anyone else chimes in who is in love with these scripts...

@garydgregory
Copy link
Member

I mean...personally, I like the wrapper scripts. But if you want, I can change this PR to simply remove the wrapper :) Should I do that?

Sure, go for it. TY.

@beatbrot
Copy link
Contributor Author

@garydgregory Done :)

@garydgregory garydgregory changed the title Update maven wrapper to 3.9.6 Remove out of date jar and scripts Feb 21, 2024
@garydgregory garydgregory merged commit 42ccfa8 into apache:master Feb 21, 2024
15 checks passed
asfgit pushed a commit that referenced this pull request Feb 21, 2024
@beatbrot beatbrot deleted the mvn-wrapper branch February 21, 2024 18:21
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.

2 participants