Skip to content

fix(gradle): fix gradle tests #30879

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix(gradle): fix gradle tests #30879

wants to merge 1 commit into from

Conversation

xiongemi
Copy link
Collaborator

@xiongemi xiongemi commented Apr 25, 2025

Current Behavior

currently, because it requires to sign locally, so i thought run command like
:project-graph:publishToMavenLocal -x :project-graph:signNxProjectGraphPluginPluginMarkerMavenPublication -x :project-graph:signPluginMavenPublication -x :project-graph:publishNxProjectGraphPluginPluginMarkerMavenPublicationToMavenLocal -x :project-graph:publishPluginMavenPublicationToMavenLocal would publish the plugin locally, but it actually does not. it does not throw an error, but does not do anything at all.
so for e2e tests, it is actually pulling the latest published gradle plugin from maven rather than test local code, hence the e2e errors.

also, currently project graph build for java version 21, we change it to java 17 to be used by ocean repo.

Expected Behavior

change the command to ./gradlew :project-graph:publishToMavenLocal -PskipSign=true and not apply signing when skip sign is true, so this should be able to publish plugin to local repository.

work with java 17

Related Issue(s)

Fixes #

Copy link

vercel bot commented Apr 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview May 1, 2025 4:55am

Copy link

nx-cloud bot commented Apr 25, 2025

View your CI Pipeline Execution ↗ for commit 96ea274.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-ci ✅ Succeeded 10m 58s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 15s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 1s View ↗
nx documentation ✅ Succeeded 32s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-01 05:07:43 UTC

@xiongemi xiongemi requested review from a team and vsavkin as code owners April 25, 2025 18:32
@xiongemi xiongemi force-pushed the fix-gradle-test branch 2 times, most recently from 25c90cf to 9c78603 Compare May 1, 2025 00:45
@xiongemi xiongemi force-pushed the fix-gradle-test branch from 9c78603 to 96ea274 Compare May 1, 2025 04:51
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