Skip to content

Add Maven Wrapper and standardize Maven invocation across CI/docs#94

Merged
infeo merged 3 commits into
developfrom
copilot/add-maven-wrapper
Jun 3, 2026
Merged

Add Maven Wrapper and standardize Maven invocation across CI/docs#94
infeo merged 3 commits into
developfrom
copilot/add-maven-wrapper

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 3, 2026

This PR adds Maven Wrapper support to integrations-api, aligned with the approach used in cryptofs, so builds no longer depend on a preinstalled system Maven. CI workflows and project docs are updated to use the wrapper consistently.

  • Wrapper bootstrap

    • Added mvnw, mvnw.cmd, and .mvn/wrapper/maven-wrapper.properties.
    • Pinned wrapper config to Maven 3.9.16 with SHA-256 verification.
  • CI command standardization

    • Replaced mvn with ./mvnw in:
      • .github/workflows/build.yml
      • .github/workflows/codeql-analysis.yml
    • Includes version-setting, project-version evaluation, compile/verify, and deploy invocations.
  • Developer-facing build docs

    • Updated README.md build section to use wrapper-based invocation and list JDK requirement.

Example command transition:

# before
mvn -B verify --no-transfer-progress

# after
./mvnw -B verify --no-transfer-progress

Updated Maven distribution URL and SHA256 checksum.
@infeo infeo marked this pull request as ready for review June 3, 2026 14:42
@infeo infeo added this to the next milestone Jun 3, 2026
@infeo infeo merged commit bb7dafc into develop Jun 3, 2026
6 checks passed
@infeo infeo deleted the copilot/add-maven-wrapper branch June 3, 2026 14:57
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