Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
/nxPublic

feat(maven): introduce maven plugin WIP#32947

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
FrozenPandaz wants to merge302 commits intomaster
base:master
Choose a base branch
Loading
frommaven-plugin
Draft

Conversation

FrozenPandaz
Copy link
Collaborator

This is still WIP

Current Behavior

There is no first-party Maven support for Nx

Expected Behavior

Maven Support is ready for usage.

Related Issue(s)

Fixes #

@vercelVercel
Copy link

vercelbot commentedOct 3, 2025
edited
Loading

The latest updates on your projects. Learn more aboutVercel for GitHub.

ProjectDeploymentPreviewUpdated (UTC)
nx-devReadyReadyPreviewOct 7, 2025 8:22pm

@netlifyNetlify
Copy link

netlifybot commentedOct 3, 2025
edited
Loading

Deploy Preview fornx-docs ready!

NameLink
🔨 Latest commitb96461c
🔍 Latest deploy loghttps://app.netlify.com/projects/nx-docs/deploys/68e570b10d346700087455c5
😎 Deploy Previewhttps://deploy-preview-32947--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to yourNetlify project configuration.

@FrozenPandazFrozenPandaz marked this pull request as draftOctober 3, 2025 14:30
@nx-cloudNx Cloud
Copy link
Contributor

nx-cloudbot commentedOct 3, 2025
edited
Loading

View yourCI Pipeline Execution ↗ for commitb96461c

CommandStatusDurationResult
nx documentation❌ Failed4m 58sView ↗

☁️Nx Cloud last updated this comment at2025-10-07 20:35:00 UTC

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

@github-actionsGitHub Actions
Copy link
Contributor

Failed to publish a PR release of this pull request, triggered by@FrozenPandaz.
See the failed workflow run at:https://github.com/nrwl/nx/actions/runs/18285305766

@github-actionsGitHub Actions
Copy link
Contributor

Failed to publish a PR release of this pull request, triggered by@FrozenPandaz.
See the failed workflow run at:https://github.com/nrwl/nx/actions/runs/18285666201

@github-actionsGitHub Actions
Copy link
Contributor

Failed to publish a PR release of this pull request, triggered by@FrozenPandaz.
See the failed workflow run at:https://github.com/nrwl/nx/actions/runs/18285926077

@github-actionsGitHub Actions
Copy link
Contributor

Failed to publish a PR release of this pull request, triggered by@FrozenPandaz.
See the failed workflow run at:https://github.com/nrwl/nx/actions/runs/18288244241

@github-actionsGitHub Actions
Copy link
Contributor

Failed to publish a PR release of this pull request, triggered by@FrozenPandaz.
See the failed workflow run at:https://github.com/nrwl/nx/actions/runs/18288244241

nx-cloud[bot]

This comment was marked as outdated.

Copy link
Contributor

@nx-cloudnx-cloudbot left a comment

Choose a reason for hiding this comment

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

Nx Cloud has identified a possible root cause for your failed CI:

Failure Classification: environment_state

The nx-maven-plugin:install task failed with "/bin/sh: 1: mvn: not found", indicating that the Maven executable is not available in the CI/CD build environment's PATH.

Analysis Details:

  1. The PR introduces a new Maven plugin with extensive Maven build infrastructure
  2. A Maven wrapper (mvnw) exists at the repository root and was added in this PR
  3. The project.json configuration for nx-maven-plugin uses "mvn install" directly
  4. The build environment does not have Maven installed in PATH

Why This Is environment_state (Not code_change):

  • The error is not caused by any code logic in the PR
  • This is a missing system dependency in the execution environment
  • The Maven wrapper (mvnw) exists and should be used, but the environment is not configured to use it
  • No similar failures in other branches suggest this is environment-specific
  • The code changes themselves are valid; the issue is purely environmental

Recommendation:
The build environment needs either:

  1. Maven installed in PATH, OR
  2. The project.json should be updated to use "./mvnw" instead of "mvn", OR
  3. The CI/CD pipeline should be configured to set up Maven before running the build

This is not a code fix that should be implemented as part of this PR review. The failure is due to environment configuration, not the PR's code changes.

A code change would likely not resolve this issue, so no action was taken.

Nx CloudView in Nx Cloud ↗


⚙️ An Nx Cloud workspace admin can disable these reviewsin workspace settings.

@github-actionsGitHub Actions
Copy link
Contributor

Failed to publish a PR release of this pull request, triggered by@FrozenPandaz.
See the failed workflow run at:https://github.com/nrwl/nx/actions/runs/18291507296

Adds automatic detection of Maven projects during nx init by looking for Maven wrapper files (mvnw, mvnw.bat) and pom.xml files. When detected, suggests installing the @nx/maven plugin, similar to how Gradle projects are detected.
Restructures the Java documentation sidebar to have separate subsections for Gradle and Maven plugins, instead of only showing Gradle at the top level. This provides equal visibility to both build tools.
- Move Gradle-specific docs from java/introduction.mdoc to java/gradle/introduction.mdoc- Create new general Java overview at java/introduction.mdoc covering both build tools- Remove special case remapping of gradle to java in plugin-mappings.ts- Both Gradle and Maven now have equal footing in the documentation structureThis addresses the TODO comment about not having Gradle as the default Java docs now that Maven is available.
Fix broken links to Gradle and Maven introduction pages by using the full path including /introduction suffix.
@github-actionsGitHub Actions
Copy link
Contributor

🐳 We have a release for that!

This PR has a release associated with it. You can try it out using this command:

npx create-nx-workspace@0.0.0-pr-32947-aa25f06 my-workspace

Or just copy this version and use it in your own command:

0.0.0-pr-32947-aa25f06
Release details📑
Published version0.0.0-pr-32947-aa25f06
Triggered by@FrozenPandaz
Branchmaven-plugin
Commitaa25f06
Workflow run18291507296

To request a new release for this pull request, mention someone from the Nx team or the@nrwl/nx-pipelines-reviewers.

Add index.mdoc landing pages for the Gradle and Maven subdirectories to resolve broken links to /docs/technologies/java/gradle and /docs/technologies/java/maven.
@github-actionsGitHub Actions
Copy link
Contributor

Failed to publish a PR release of this pull request, triggered by@FrozenPandaz.
See the failed workflow run at:https://github.com/nrwl/nx/actions/runs/18293215975

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@nx-cloudnx-cloud[bot]nx-cloud[bot] left review comments

@jaysoojaysooAwaiting requested review from jaysoojaysoo is a code owner

@AgentEnderAgentEnderAwaiting requested review from AgentEnderAgentEnder is a code owner

@vsavkinvsavkinAwaiting requested review from vsavkinvsavkin is a code owner

@meeroslavmeeroslavAwaiting requested review from meeroslavmeeroslav is a code owner

@JamesHenryJamesHenryAwaiting requested review from JamesHenryJamesHenry is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@FrozenPandaz@barbados-clemens

[8]ページ先頭

©2009-2025 Movatter.jp