Skip to content

Disable screenshots until time out bug is fixed#300293

Merged
hediet merged 1 commit intomainfrom
hediet/b/splendid-antelope
Mar 10, 2026
Merged

Disable screenshots until time out bug is fixed#300293
hediet merged 1 commit intomainfrom
hediet/b/splendid-antelope

Conversation

@hediet
Copy link
Copy Markdown
Member

@hediet hediet commented Mar 9, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 9, 2026 23:26
@hediet hediet enabled auto-merge (squash) March 9, 2026 23:26
@hediet hediet self-assigned this Mar 9, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.112.0 milestone Mar 9, 2026
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

Disables the Component Screenshots GitHub Actions job while a timeout-related issue is being investigated, preventing the screenshot workflow from running its steps in CI.

Changes:

  • Adds a job-level condition to skip the screenshots job unconditionally.


jobs:
screenshots:
if: false # temporarily disabled
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

jobs.<job_id>.if is expected to be a GitHub Actions expression string. Using the YAML boolean false here can be parsed as a boolean value (not an expression) and may cause the workflow to be rejected or behave unexpectedly. Consider switching to an explicit expression (e.g. if: ${{ false }}) and/or quoting the value, and include a tracking reference (issue/PR) in the comment since this is meant to be temporary.

Suggested change
if: false # temporarily disabled
if: ${{ false }} # temporarily disabled (TODO: add tracking issue/PR reference)

Copilot uses AI. Check for mistakes.
@hediet hediet merged commit 74c7650 into main Mar 10, 2026
23 checks passed
@hediet hediet deleted the hediet/b/splendid-antelope branch March 10, 2026 00:52
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