-
Notifications
You must be signed in to change notification settings - Fork 1.2k
docs: fix outdated Maven subtitle in INSTALL.md #11012
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
Conversation
Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md)
|
Hi team, I’m a new joiner to the community and am currently building CloudStack locally by following the instructions in INSTALL.md. While going through the setup, I noticed this minor inconsistency and thought it’d be a good opportunity to make my first contribution. I'm looking forward to contributing more and learning from the project and the community! Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
welcome @he1l0world , hope you enjoy it. thanks for the doc-update
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11012 +/- ##
============================================
- Coverage 16.60% 16.60% -0.01%
Complexity 13924 13924
============================================
Files 5730 5730
Lines 508082 508082
Branches 61770 61770
============================================
- Hits 84388 84386 -2
- Misses 414259 414260 +1
- Partials 9435 9436 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Thank you! @DaanHoogland |
Thanks for the review and approval! Since I don't have the merge permissions, kindly help with the merge when you get a chance. Thank you! @DaanHoogland |
Awesome work, congrats on your first merged pull request! |
Description
This PR updates the Maven setup subtitle in INSTALL.md to reflect the correct version (3.9.9), aligning it with the updated download URL introduced in PR #10800.
In that PR, the download link for Maven was correctly updated from 3.6.0 to 3.9.9, but the subtitle above the instructions still referenced the older version (3.6.0). This follow-up change ensures consistency and clarity.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
Tested by reviewing the Markdown.