Skip to content

Conversation

@wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Aug 28, 2025

No description provided.

Copilot AI review requested due to automatic review settings August 28, 2025 20:27
@wtgodbe wtgodbe requested a review from a team as a code owner August 28, 2025 20:27
@github-actions github-actions bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Aug 28, 2025
@wtgodbe wtgodbe merged commit ff3b44a into main Aug 28, 2025
6 of 25 checks passed
@wtgodbe wtgodbe deleted the wtgodbe/FixUnofficial branch August 28, 2025 20:28
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 fixes the unofficial build pipeline by correcting a YAML syntax error and standardizing submodule initialization across all build jobs.

  • Fixed syntax error: changed esources: to resources: in the YAML configuration
  • Added consistent submodule initialization (git submodule update --init) to all build jobs
  • Removed the source build job section entirely

Comment on lines 691 to 692
- script: git submodule update --init
displayName: Update submodules
Copy link

Copilot AI Aug 28, 2025

Choose a reason for hiding this comment

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

This job already has a duplicate submodule update step in the beforeBuild section (lines 688-689). The redundant step in the steps section should be removed to avoid duplication.

Suggested change
- script: git submodule update --init
displayName: Update submodules

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants