Skip to content

fix: update wasm golden files to include GH_HOST env var in Copilot CLI install step#24330

Merged
pelikhan merged 1 commit intomainfrom
copilot/fix-build-wasm
Apr 3, 2026
Merged

fix: update wasm golden files to include GH_HOST env var in Copilot CLI install step#24330
pelikhan merged 1 commit intomainfrom
copilot/fix-build-wasm

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 3, 2026

Summary

Fixes the failing TestWasmGolden_CompileFixtures tests (basic-copilot and with-imports).

The golden files were out of date — a recent change added GH_HOST: github.com to the env block of the "Install GitHub Copilot CLI" step, but the golden files weren't updated to match.

Changes

Updated golden files:

  • pkg/workflow/testdata/TestWasmGolden_CompileFixtures/basic-copilot.golden
  • pkg/workflow/testdata/TestWasmGolden_CompileFixtures/with-imports.golden

Both files now include:

        env:
          GH_HOST: github.com

on the Install GitHub Copilot CLI step.

Reference: https://github.com/github/gh-aw/actions/runs/23950605290/job/69857142842

…LI install step

Agent-Logs-Url: https://github.com/github/gh-aw/sessions/f24f32e4-5c2f-4cde-a97c-27365f013aa0

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI requested a review from pelikhan April 3, 2026 15:10
@pelikhan pelikhan marked this pull request as ready for review April 3, 2026 15:11
Copilot AI review requested due to automatic review settings April 3, 2026 15:11
@pelikhan pelikhan merged commit 211dcda into main Apr 3, 2026
51 checks passed
@pelikhan pelikhan deleted the copilot/fix-build-wasm branch April 3, 2026 15:11
Copy link
Copy Markdown
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

Updates wasm compile-fixture golden outputs to match the current generated workflow YAML, specifically reflecting the newly added GH_HOST: github.com environment variable on the “Install GitHub Copilot CLI” step.

Changes:

  • Updated basic-copilot.golden to include env: GH_HOST: github.com on the Copilot CLI install step.
  • Updated with-imports.golden to include env: GH_HOST: github.com on the Copilot CLI install step.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pkg/workflow/testdata/TestWasmGolden_CompileFixtures/basic-copilot.golden Aligns golden output with generated workflow by adding GH_HOST: github.com to the Copilot CLI install step env.
pkg/workflow/testdata/TestWasmGolden_CompileFixtures/with-imports.golden Same golden update as above for the with-imports fixture output.

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

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.

3 participants