-
Notifications
You must be signed in to change notification settings - Fork 902
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
[BUILD] Bump Maven from 3.8.8 to 3.9.9 #6664
Conversation
Other popular projects are now using Maven 3.9, e.g. Spark , and Celeborn. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6664 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 683 683
Lines 42205 42205
Branches 5756 5756
======================================
Misses 42205 42205 ☔ View full report in Codecov by Sentry. |
Thanks, merged to master (1.10.0). |
Seems we have used a different variable in GitHub workflows to gain the same purpose, it would be great to migrate to the official approch. https://maven.apache.org/configure.html#maven_args-environment-variable
|
Cool. Sounds that |
🔍 Description
Issue References 🔗
This pull request fixes #
Describe Your Solution 🔧
Types of changes 🔖
Test Plan 🧪
Behavior Without This Pull Request ⚰️
Behavior With This Pull Request 🎉
Related Unit Tests
Checklist 📝
Be nice. Be informative.