Skip to content

Add a note to explicitly state that signed pipeline are not supported … - #3155

Closed
ss1909 wants to merge 1 commit into
mainfrom
sneha/A-1677-explicit-not_supported_signed_pipeline_HA
Closed

Add a note to explicitly state that signed pipeline are not supported …#3155
ss1909 wants to merge 1 commit into
mainfrom
sneha/A-1677-explicit-not_supported_signed_pipeline_HA

Conversation

@ss1909

@ss1909 ss1909 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot added the agent label Aug 18, 2026
@ss1909 ss1909 changed the title Add a not to explicitly state that signed pipeline are not supported … Add a note to explicitly state that signed pipeline are not supported … Aug 18, 2026
@ss1909
ss1909 marked this pull request as ready for review August 18, 2026 03:46
@ss1909
ss1909 requested a review from a team as a code owner August 18, 2026 03:46

@buildsworth-bk-app buildsworth-bk-app Bot 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.

The hosted-agent overview now exposes the limitation, but the change leaves the second documentation entry point from A-1677 unchanged. The remaining copy and formatting issues are inline.

Want to dig deeper?

Paste this into your agent to explore the findings from this review's Buildkite build:

Download the buildsworth logs from build 17368, then answer my questions about the findings.

Install the reading-buildsworth-logs skill to run this.

About buildsworth

Model: gpt-5.6-sol with xhigh thinking.

How to request a review: Comment @buildsworth-bk review on the PR, or request buildsworth-bk as a reviewer.

Risk labels (how buildsworth classifies risk) — buildsworth classifies risk itself from the diff. To let it approve, grant L2 approval by mentioning @buildsworth-bk (see L2 approval grant):

  • L1 — Low risk (dep bumps, docs/copy, lockfiles, small presentational fixes). buildsworth may approve by default.
  • L2 — Standard risk (new UI, additive API fields, refactors). Approved only with an L2 grant; otherwise comment-only.
  • L3 — High risk (auth, migrations, payments, secrets, perf-critical paths). Human review always required.

Comment thread pages/agent/buildkite_hosted.md Outdated
@ss1909
ss1909 force-pushed the sneha/A-1677-explicit-not_supported_signed_pipeline_HA branch 2 times, most recently from 0041c1e to 59f141f Compare August 18, 2026 04:05

@buildsworth-bk-app buildsworth-bk-app Bot 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.

The new sentence covers the signed-pipelines page and fixes the agreement issue. The formatting part of the original blocking thread remains open.

Want to dig deeper?

Paste this into your agent to explore the findings from this review's Buildkite build:

Download the buildsworth logs from build 17388, then answer my questions about the findings.

Install the reading-buildsworth-logs skill to run this.

@ss1909
ss1909 force-pushed the sneha/A-1677-explicit-not_supported_signed_pipeline_HA branch from 59f141f to 48ebf3e Compare August 18, 2026 04:36
@buildsworth-bk-app
buildsworth-bk-app Bot dismissed their stale review August 18, 2026 04:39

Previous concerns have been addressed.

@ss1909 ss1909 closed this Aug 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant