Skip to content

Conversation

@jbogard
Copy link
Contributor

@jbogard jbogard commented Nov 3, 2025

Swaps the release pipelines to push from the Windows images so that the Windows assemblies are packed and published

@jbogard jbogard requested a review from Copilot November 3, 2025 19:51
Copy link

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 reorganizes the GitHub Actions workflow structure to improve job execution order and standardize indentation for YAML files. The changes swap the order of the build and build-windows jobs so that the Linux build runs first, followed by the Windows build which now depends on it. Additionally, the build process has been standardized to use ./Build.ps1 across both workflows.

  • Reordered jobs in both CI and release workflows to run Linux builds first, with Windows builds depending on them
  • Standardized build commands to use ./Build.ps1 consistently across all jobs
  • Added YAML indentation configuration to .editorconfig

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/release.yml Swapped job order to run build (Linux) first, then build-windows; added YAML document separator; standardized build command
.github/workflows/ci.yml Swapped job order to run build (Linux) first, then build-windows; standardized build command
.editorconfig Added indentation configuration for YAML files (2 spaces)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link

github-actions bot commented Dec 7, 2025

This pull request has been automatically locked since there has not been any recent activity after it was closed.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants