fix(gradle): use tooling api compatible flags#34247
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Deploy Preview for nx-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
View your CI Pipeline Execution ↗ for commit 858d9f4
☁️ Nx Cloud last updated this comment at |
3b956a3 to
e5d0f7f
Compare
6fbc036 to
3f720f3
Compare
357b985 to
858d9f4
Compare
There was a problem hiding this comment.
Important
At least one additional CI pipeline execution has run since the conclusion below was written and it may no longer be applicable.
Nx Cloud has identified a possible root cause for your failed CI:
Our build encountered a Gradle daemon cache cleanup failure where the daemon attempted to delete compilation cache directories while still holding file locks. This is an infrastructure-level timing issue with daemon cache management, not a code logic error from the PR changes.
No code changes were suggested for this issue.
Trigger a rerun:
🎓 Learn more about Self-Healing CI on nx.dev
<!-- Please make sure you have read the submission guidelines before posting an PR --> <!-- https://github.com/nrwl/nx/blob/master/CONTRIBUTING.md#-submitting-a-pr --> <!-- Please make sure that your commit message follows our format --> <!-- Example: `fix(nx): must begin with lowercase` --> <!-- If this is a particularly complex change or feature addition, you can request a dedicated Nx release for this pull request branch. Mention someone from the Nx team or the `@nrwl/nx-pipelines-reviewers` and they will confirm if the PR warrants its own release for testing purposes, and generate it for you if appropriate. --> ## Current Behavior <!-- This is the behavior we have today --> ## Expected Behavior <!-- This is the behavior we should expect with the changes in this PR --> --rerun is not tooling api compatible and therefore will break usage of the batch executor. Replaced the flag with --rerun-tasks. ## Related Issue(s) <!-- Please link the issue being fixed so it gets closed when this is merged. --> Fixes # (cherry picked from commit 5a424fa)
|
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
Expected Behavior
--rerun is not tooling api compatible and therefore will break usage of the batch executor. Replaced the flag with --rerun-tasks.
Also updated the build and test listeners so that we normalize the task paths that we use between Gradle and Nx. There were cases where gradle prepended
:to its task paths, which did not match what Nx was using.Related Issue(s)
Fixes #