Adds a GitHub Actions workflow to run CodeQL security analysis on C# projects.#2
Merged
Adds a GitHub Actions workflow to run CodeQL security analysis on C# projects.#2
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
Adds a GitHub Actions workflow to run CodeQL security analysis on C# projects.
- Introduces a new
.github/workflows/codeql.ymlfile. - Triggers analysis on
pushandpull_requestevents formainanddevelopbranches. - Implements steps to checkout code, set up .NET 8.0, initialize CodeQL, build the project, and perform analysis.
Comments suppressed due to low confidence (2)
.github/workflows/codeql.yml:25
- Add
with: fetch-depth: 0to the checkout step so that CodeQL has access to the full Git history and tags, as recommended in the CodeQL documentation.
uses: actions/checkout@v4
.github/workflows/codeql.yml:3
- [nitpick] Consider adding a
scheduletrigger (e.g.,- cron: "0 0 * * 0") underonto run CodeQL analysis regularly even when there are no code changes.
on:
Changed the dotnet-version in the CodeQL GitHub Actions workflow from 8.0.x to 9.0.x to align with the project's SDK requirements.
ambvdijk
changed the title
Updated the checkout step in the CodeQL workflow to use 'fetch-depth: 0', ensuring the full git history is available for analysis.
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
|
ambvdijk
added a commit
that referenced
this pull request
Jul 9, 2025
* Adds a GitHub Actions workflow to run CodeQL security analysis on C# projects. (#2) * Create codeql.yml * Update CodeQL workflow to use .NET 9.0.x Changed the dotnet-version in the CodeQL GitHub Actions workflow from 8.0.x to 9.0.x to align with the project's SDK requirements. * Set fetch-depth to 0 in CodeQL workflow Updated the checkout step in the CodeQL workflow to use 'fetch-depth: 0', ensuring the full git history is available for analysis. * Add develop branch to CI workflow triggers (#1) Updated dotnet and sonarqube GitHub Actions workflows to trigger on both main and develop branches for push and pull request events. This ensures CI processes run for changes targeting the develop branch. * Remove --no-restore from dotnet build step (#3) The build step in the GitHub Actions workflow now runs 'dotnet build' without the '--no-restore' flag, ensuring dependencies are restored during the build process. * Fix incorrect syntax in dotnet.yml (#4) * Remove --no-restore from dotnet build step The build step in the GitHub Actions workflow now runs 'dotnet build' without the '--no-restore' flag, ensuring dependencies are restored during the build process. * Update dotnet.yml * Add Dependabot configuration for NuGet updates (#5) Introduces a dependabot.yml file to enable automated weekly updates for NuGet dependencies in the repository. * Add refactored CommandHandlerBase unit tests Introduces a modular and SOLID-compliant test suite for CommandHandlerBase, including constructor validation, edge case, and shared test infrastructure. Adds new test classes, helpers, and a detailed README describing the new structure. Also adds Moq as a test dependency. * Add unit tests for CommandHandlerBase.HandleAsync Introduces tests covering core functionality of the HandleAsync method in CommandHandlerBase, including event envelope creation, ULID generation, timestamp handling, serialization, storage, and abstract method calls. * Update CI workflows for improved compatibility Switch SonarQube workflow from Windows to Ubuntu, update path separators for Unix compatibility, and remove PowerShell-specific commands. Minor formatting improvements were made to the dotnet workflow for consistency. * Update SonarQube workflow for .NET 9 and code coverage Adds setup for .NET 9.0, configures SonarQube scanner to use OpenCover reports, and updates test command to collect code coverage in OpenCover format. These changes improve code analysis and ensure compatibility with the latest .NET version. * Update MathMax.EventSourcing.UnitTests/MathMax.EventSourcing.UnitTests.csproj Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Make CreateTestCommand static and update test assertion Changed CreateTestCommand to a static method in CommandHandlerBaseTestFixture for easier access without instantiation. Updated a test in CommandHandlerBaseConstructorTests to use Assert.IsType with exactMatch: false instead of Assert.IsAssignableFrom. * Refactor test command creation in unit tests Replaced usage of _fixture.CreateTestCommand() with CommandHandlerBaseTestFixture.CreateTestCommand() in CommandHandlerBaseEdgeCaseTests and CommandHandlerBaseHandleAsyncTests for consistency and clarity. * Refactor test helpers into separate files Split TestHelpers.cs into individual files for each test class: TestCommand, TestEvent, TestCommandHandler, SpyTestCommandHandler, TestCommandHandlerWithNullAggregateId, and TestCommandHandlerWithNullVersion. This improves code organization and maintainability. --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
ambvdijk
added a commit
that referenced
this pull request
Jul 9, 2025
* Adds a GitHub Actions workflow to run CodeQL security analysis on C# projects. (#2) * Create codeql.yml * Update CodeQL workflow to use .NET 9.0.x Changed the dotnet-version in the CodeQL GitHub Actions workflow from 8.0.x to 9.0.x to align with the project's SDK requirements. * Set fetch-depth to 0 in CodeQL workflow Updated the checkout step in the CodeQL workflow to use 'fetch-depth: 0', ensuring the full git history is available for analysis. * Add develop branch to CI workflow triggers (#1) Updated dotnet and sonarqube GitHub Actions workflows to trigger on both main and develop branches for push and pull request events. This ensures CI processes run for changes targeting the develop branch. * Remove --no-restore from dotnet build step (#3) The build step in the GitHub Actions workflow now runs 'dotnet build' without the '--no-restore' flag, ensuring dependencies are restored during the build process. * Fix incorrect syntax in dotnet.yml (#4) * Remove --no-restore from dotnet build step The build step in the GitHub Actions workflow now runs 'dotnet build' without the '--no-restore' flag, ensuring dependencies are restored during the build process. * Update dotnet.yml * Add Dependabot configuration for NuGet updates (#5) Introduces a dependabot.yml file to enable automated weekly updates for NuGet dependencies in the repository.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Overview
Adds a GitHub Actions workflow to run CodeQL security analysis on C# projects.
.github/workflows/codeql.ymlfile.pushandpull_requestevents formainanddevelopbranches.