- Notifications
You must be signed in to change notification settings - Fork2.6k
feat(dotnet): add initial draft of .NET plugin#32869
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
base:master
Are you sure you want to change the base?
Conversation
vercelbot commentedSep 26, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
The latest updates on your projects. Learn more aboutVercel for GitHub.
|
netlifybot commentedSep 30, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✅ Deploy Preview fornx-docs ready!
To edit notification comments on pull requests, go to yourNetlify project configuration. |
nx-cloudbot commentedOct 1, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
View yourCI Pipeline Execution ↗ for commitc0f37d0
☁️Nx Cloud last updated this comment at |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
nx-cloudbot left a comment• edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Nx Cloud is proposing a fix for your failed CI:
We've fixed the test failure by removing the erroneousexec --
prefix from therunCommand
call. This was causing the Infrastructure → Core project reference to fail during setup, which prevented the dependency detection test from passing. The command now matches the pattern used by the other project reference commands in the test.
We verified this fix by re-runninge2e-dotnet:e2e-ci--src/dotnet-multi-project.test.ts
.
diff --git a/e2e/dotnet/src/dotnet-multi-project.test.ts b/e2e/dotnet/src/dotnet-multi-project.test.tsindex 1fff895351..74e9b67919 100644--- a/e2e/dotnet/src/dotnet-multi-project.test.ts+++ b/e2e/dotnet/src/dotnet-multi-project.test.ts@@ -47,7 +47,7 @@ describe('.NET Plugin - Multi-Project Scenarios', () => { // Set up project references to create a dependency chain runCommand(- `exec -- dotnet add Infrastructure/Infrastructure.csproj reference Core/Core.csproj`+ `dotnet add Infrastructure/Infrastructure.csproj reference Core/Core.csproj` ); runCommand( `dotnet add Application/Application.csproj reference Core/Core.csproj`
Apply fix locally ↗ View interactive diff ↗
⚙️ An Nx Cloud workspace admin can disable these reviewsin workspace settings.
Failed to publish a PR release of this pull request, triggered by@AgentEnder. |
Current Behavior
Expected Behavior
Related Issue(s)
Fixes #