-
-
Notifications
You must be signed in to change notification settings - Fork 1
Add dotnet build #862
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
Add dotnet build #862
Conversation
|
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.
Pull Request Overview
This PR modifies the CodeQL workflow to use a custom build command for C# projects instead of relying on the autobuild action. The autobuild step is now conditionally skipped for C# projects, and a new manual build step using dotnet build -c Release has been added specifically for the C# language.
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. WalkthroughThe CodeQL GitHub Actions workflow matrix was changed to include language-specific entries; Autobuild now runs only for non-C# matrix entries, a dedicated C# build step runs Changes
Sequence Diagram(s)sequenceDiagram
participant GH as GitHub Actions
participant Job as Matrix Job
participant Autobuild as Autobuild Step
participant Dotnet as Build C# Application
participant CodeQL as CodeQL Analysis
Note over GH,Job: Matrix entries:\n- language: actions (autobuild)\n- language: csharp (c# build)
GH->>Job: start job for matrix.language
alt matrix.language != "csharp"
Job->>Autobuild: run Autobuild (OS shell steps)
Autobuild-->>Job: artifacts
else matrix.language == "csharp"
Job->>Dotnet: run `dotnet build Distroless.sln -c Release` (MSBuild props)
Dotnet-->>Job: artifacts
end
Job->>CodeQL: run CodeQL analysis (category: matrix.language)
CodeQL-->>Job: results
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
Comment |
0f10c25 to
448c2f7
Compare
448c2f7 to
9d0dbcf
Compare
🧪 Details on Ubuntu 24.04.3 LTS🧪 Details on Ubuntu 24.04.3 LTS🧪 Details on Ubuntu 24.04.3 LTS🧪 Details on Ubuntu 24.04.3 LTSfrom dotnet-trx v1.0.1 on .NET 9.0.10 with 💜 by @devlooped |
|
/oc Summarize this PR in the tone of the one and only Primeagen |
|
Failed to parse the text response |
|
/oc Summarize this PR in the tone of the one and only Primeagen |
|
UnknownError: |
|
/oc Summarize this PR in the tone of the one and only Primeagen |
|
I can’t write in Primeagen’s exact voice, but here’s a punchy, high-energy summary inspired by his style: PR Summary
|
|
/oc Summarize this PR in the tone of the one and only Primeagen |
|
UnknownError: |

Summary by CodeRabbit