Skip to content

Clarify required status check guidance#102

Merged
trask merged 2 commits into
open-telemetry:mainfrom
trask:clarify-required-check-guidance
Jul 20, 2026
Merged

Clarify required status check guidance#102
trask merged 2 commits into
open-telemetry:mainfrom
trask:clarify-required-check-guidance

Conversation

@trask

@trask trask commented Jul 18, 2026

Copy link
Copy Markdown
Member

Use count-neutral wording for required status check failures in dashboard-managed PR comments. A repository can expose one required aggregate check backed by multiple failing jobs, so singular wording can understate the work that needs investigation. Bump the status comment revision to refresh existing comments.

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

Clarifies CI failure guidance without implying a specific failure count and refreshes existing dashboard comments.

Changes:

  • Uses count-neutral required-check guidance.
  • Updates tests and increments the comment revision.

Reviewed changes

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

File Description
pr_status_comment.py Updates guidance and revision.
test_pr_status_comment.py Verifies the revised wording.

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

@trask
trask marked this pull request as ready for review July 18, 2026 13:12
@trask
trask requested a review from a team as a code owner July 18, 2026 13:12
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 20, 2026

Copy link
Copy Markdown

Pull request dashboard status

  • Status: Merged.

This automated status or its linked feedback items may be incorrect. If something looks wrong, report it with the result you expected.

…eck-guidance

# Conflicts:
#	.github/scripts/pull-request-dashboard/pr_status_comment.py

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

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

@trask
trask merged commit d8766a2 into open-telemetry:main Jul 20, 2026
5 checks passed
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