Skip to content

Dev#8

Merged
julerobb1 merged 7 commits intodevfrom
main
Jul 9, 2025
Merged

Dev#8
julerobb1 merged 7 commits intodevfrom
main

Conversation

@julerobb1
Copy link
Owner

No description provided.

@julerobb1 julerobb1 requested a review from Copilot July 8, 2025 16:06
Copy link
Contributor

Copilot AI left a 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 adds a GitHub Actions workflow to automate restoring, building, and testing the Processor-Emulator .NET solution on pushes and pull requests to main/master.

  • Introduces a .NET Build CI pipeline with checkout, setup, restore, build, and test steps.
  • Configures Windows runners and uses .NET 6.0.
Comments suppressed due to low confidence (1)

.github/workflows/dotnet.yml:29

  • The continue-on-error: true setting will prevent test failures from failing the workflow. Consider removing this or setting it to false so that failing tests block the build.
      continue-on-error: true

runs-on: windows-latest

steps:
- uses: actions/checkout@v4
Copy link

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] Consider adding a caching step (e.g., using actions/cache) for NuGet packages to speed up restore times in the workflow.

Copilot uses AI. Check for mistakes.
@julerobb1
Copy link
Owner Author

This pull request adds a new GitHub Actions workflow to automate the build, test, and dependency restoration processes for the .NET project. The workflow is triggered on pushes and pull requests to the main or master branches.

CI/CD Workflow Addition:

  • .github/workflows/dotnet.yml: Introduced a .NET Build workflow that sets up the .NET environment, restores dependencies, builds the solution (Processor-Emulator.sln), and runs tests with error continuation enabled.

@julerobb1 julerobb1 merged commit 8b280ba into dev Jul 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant