Skip to content

Handle unknown checks case#8827

Merged
alexr00 merged 1 commit into
mainfrom
alexr00/surprised-moth
Jul 3, 2026
Merged

Handle unknown checks case#8827
alexr00 merged 1 commit into
mainfrom
alexr00/surprised-moth

Conversation

@alexr00

@alexr00 alexr00 commented Jul 3, 2026

Copy link
Copy Markdown
Member

Fixes #8813

Copilot AI review requested due to automatic review settings July 3, 2026 09:27
@alexr00 alexr00 enabled auto-merge (squash) July 3, 2026 09:27
@alexr00 alexr00 self-assigned this Jul 3, 2026

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.

Pull request overview

This PR addresses issue #8813 by ensuring the PR tree does not show an “All checks passed” (green) decoration when the extension can’t reliably determine branch-protection / checks status (i.e., checks are missing or reported as unknown).

Changes:

  • Introduces an UnsatisfiedChecks.Unknown bit to represent “status could not be determined”.
  • Updates PR tree status computation to set Unknown when checks are unavailable/unknown and no other unsatisfied condition is detected.
  • Updates status-decoration mapping to account for the new Unknown flag.
Show a summary per file
File Description
src/view/prsTreeModel.ts Sets UnsatisfiedChecks.Unknown instead of treating missing/unknown checks as “no unsatisfied checks”.
src/github/utils.ts Adds UnsatisfiedChecks.Unknown to the bitmask enum.
src/github/markdownUtils.ts Adds handling for UnsatisfiedChecks.Unknown in getStatusDecoration.

Review details

  • Files reviewed: 3/3 changed files
  • Comments generated: 2
  • Review effort level: Low

Comment thread src/github/markdownUtils.ts
Comment thread src/github/markdownUtils.ts
@alexr00 alexr00 merged commit cd4fdb7 into main Jul 3, 2026
8 checks passed
@alexr00 alexr00 deleted the alexr00/surprised-moth branch July 3, 2026 09:47
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.

PR shows green checkmark for ready to commit even though branch protection policy is not met yet (review pending)

3 participants