Conversation
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.
Contributor
There was a problem hiding this comment.
Pull Request Overview
Fix incorrect syntax in the GitHub Actions workflow by properly specifying the pull_request branches filter and simplifying the build command.
- Added a
branches:key underpull_requesttrigger - Updated the build step to remove the
--no-restoreflag
Comments suppressed due to low confidence (2)
.github/workflows/dotnet.yml:12
- Indent
branches:two spaces deeper underpull_request:so it’s properly nested. It should align with the branch list entries.
branches:
.github/workflows/dotnet.yml:30
- Restore the leading
-beforerun:to mark it as a list item understeps:; it was removed by mistake.
run: dotnet build
|
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
Fix incorrect syntax in the GitHub Actions workflow by properly specifying the pull_request branches filter and simplifying the build command.
branches:key underpull_requesttrigger--no-restoreflag