Skip to content

Conversation

@mikepapadim
Copy link
Member

This commit adds Maven wrapper to the project, allowing developers to build the project without having Maven pre-installed. The Makefile has been updated to use the wrapper (./mvnw) instead of requiring a system-wide Maven installation.

Changes:

  • Add Maven wrapper scripts (mvnw, mvnw.cmd) and configuration
  • Update Makefile to use Maven wrapper via MVN variable
  • Configure wrapper to use Maven 3.9.6

🤖 Generated with Claude Code

This commit adds Maven wrapper to the project, allowing developers to build the project without having Maven pre-installed. The Makefile has been updated to use the wrapper (./mvnw) instead of requiring a system-wide Maven installation.

Changes:
- Add Maven wrapper scripts (mvnw, mvnw.cmd) and configuration
- Update Makefile to use Maven wrapper via MVN variable
- Configure wrapper to use Maven 3.9.6

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@mikepapadim mikepapadim self-assigned this Oct 27, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds Maven Wrapper support to enable building the project without requiring a pre-installed Maven installation. The wrapper downloads and uses Maven 3.9.6 automatically when invoked.

Key Changes:

  • Added Maven wrapper scripts for Unix/Linux (mvnw) and Windows (mvnw.cmd) platforms
  • Updated build automation to use the wrapper instead of system Maven
  • Configured wrapper to use Maven 3.9.6 with script-only distribution

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
mvnw Unix/Linux shell script for Maven wrapper (v3.3.4) that downloads and executes Maven
mvnw.cmd Windows batch/PowerShell script for Maven wrapper (v3.3.4) that downloads and executes Maven
Makefile Updated to use ./mvnw via MVN variable instead of system mvn command
.mvn/wrapper/maven-wrapper.properties Configuration specifying Maven 3.9.6 distribution URL and wrapper version

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mikepapadim mikepapadim merged commit 5a96280 into beehive-lab:main Oct 30, 2025
1 check passed
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.

1 participant