Skip to content

docs: add Maven Wrapper usage instructions for issue #26#79

Open
Nalwogatracy wants to merge 1 commit intobos-com:mainfrom
Nalwogatracy:fix/issue-26-maven-wrapper
Open

docs: add Maven Wrapper usage instructions for issue #26#79
Nalwogatracy wants to merge 1 commit intobos-com:mainfrom
Nalwogatracy:fix/issue-26-maven-wrapper

Conversation

@Nalwogatracy
Copy link
Copy Markdown

  • Added mvnw commands for Linux/Mac/Windows
  • Explained benefits of using Maven Wrapper
  • Added instructions to generate wrapper if missing

- Added mvnw commands for Linux/Mac/Windows
- Explained benefits of using Maven Wrapper
- Added instructions to generate wrapper if missing
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 21, 2026

@Nalwogatracy is attempting to deploy a commit to the mukisalivingstone133-9917's projects Team on Vercel.

A member of the Team first needs to authorize it.

@RonaldRonnie RonaldRonnie self-requested a review April 21, 2026 18:38
Comment thread README.md
This project includes the Maven Wrapper (`mvnw`) to ensure consistent builds across all environments without requiring Maven to be installed globally.

### Why Use Maven Wrapper?
- ✅ No need to install Maven separately
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the documentation update - this is helpful 👍
However, the issue requires adding the actual Maven Wrapper files (mvnw, mvnw.cmd, .mvn/) to the repository.

Right now, the README references ./mvnw, but those files are not present, so the commands will fail.

Please generate the wrapper using:

mvn wrapper:wrapper

and commit the generated files so the project can build without requiring a global Maven installation.

Copy link
Copy Markdown
Contributor

@RonaldRonnie RonaldRonnie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the documentation update @Nalwogatracy - this is helpful 👍
However, the issue requires adding the actual Maven Wrapper files (mvnw, mvnw.cmd, .mvn/) to the repository.

Right now, the README references ./mvnw, but those files are not present, so the commands will fail.

Please generate the wrapper using:
mvn wrapper:wrapper

and commit the generated files so the project can build without requiring a global Maven installation.

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