Visitar URL original
chore: migrate CI to GitHub Actions by rzhao271 · Pull Request #8991 · microsoft/vscode-pull-request-github · GitHub
Skip to content

chore: migrate CI to GitHub Actions - #8991

Open
Raymond Zhao (rzhao271) wants to merge 4 commits into
mainfrom
rzhao271/gh-ci
Open

chore: migrate CI to GitHub Actions#8991
Raymond Zhao (rzhao271) wants to merge 4 commits into
mainfrom
rzhao271/gh-ci

Conversation

@rzhao271

@rzhao271 Raymond Zhao (rzhao271) commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI balanced review requested due to automatic review settings September 29, 2026 21:42
@rzhao271 Raymond Zhao (rzhao271) added this to the 1.140.0 milestone Sep 29, 2026
Copilot AI reviewed Sep 29, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The replacement CI gate needs a successful run and human confirmation of test reporting; the current workflow only uploads JUnit results as an artifact.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

This PR moves pull-request CI from Azure Pipelines to GitHub Actions.

Changes:

  • Adds a macOS workflow for dependency checks, compilation, linting, and tests.
  • Removes the Azure PR pipeline and excludes GitHub workflow YAML from the indentation check.
File Description
build/​filters.js Excludes workflow YAML from indentation checks.
azure-pipeline.pr.yml Removes the former PR pipeline.
.github/​workflows/​pr.yml Adds the replacement PR workflow.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/pr.yml
@rzhao271
Plan CI test summary update
3a30d52 
Co-authored-by: rzhao271 <7199958+rzhao271@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 29, 2026 21:51
@rzhao271
Publish JUnit test summary in CI
a2f7319 
Co-authored-by: rzhao271 <7199958+rzhao271@users.noreply.github.com>
Copilot AI reviewed Sep 29, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

Replacing the PR validation gate needs human confirmation that the workflow runs and satisfies branch-protection requirements.

Review effort: Balanced
Findings: None

Resolved since last review (1)

alexr00
Alex Ross (alexr00) reviewed Sep 30, 2026
View reviewed changes

@alexr00 Alex Ross (alexr00) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What's the motivation behind this (also tests are failing)?

@rzhao271

Raymond Zhao (rzhao271) commented Sep 30, 2026

Copy link
Copy Markdown
Contributor Author

We're moving old CI projects off of https://dev.azure.com/vscode and plan to reallocate the 1ES-hosted runners elsewhere.

Copilot AI balanced review requested due to automatic review settings October 5, 2026 05:18
Copilot AI reviewed Oct 5, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

Static review found no blocking issues, and the replacement preserves validation and failure-time test reporting.

Review effort: Balanced
Findings: None

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexr00 Alex Ross (alexr00) alexr00 left review comments
Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
Copilot code review Copilot Copilot left review comments
@roblourens roblourens roblourens approved these changes
@Yoyokrazy Michael Lively (Yoyokrazy) Yoyokrazy approved these changes
@benvillalobos Ben Villalobos (benvillalobos) benvillalobos left review comments

Assignees

@rzhao271 Raymond Zhao (rzhao271)

Labels

None yet

Projects

None yet

Milestone

1.140.0

Development

Successfully merging this pull request may close these issues.

8 participants

@rzhao271 @roblourens @benvillalobos @Yoyokrazy @alexr00 @dmitrivMS