Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 5 additions & 22 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,28 +19,11 @@ List of what the PR changes. If the PR changes the CLI arguments, consider addin
Can skip if changes are simple or clear from the commit messages.
-->

## Documentation and publication

<!--
The following details help our docs writer bot decide whether and how to draft docs for this change.
-->

**Does this change need public documentation?**

- [ ] Yes
- [ ] No
- [ ] Unsure

**Is this change safe to document publicly when this PR merges?**

- [ ] Yes
- [ ] No, the documentation must be held
- [ ] It can be documented as a limited-availability or preview feature
- [ ] Not applicable (docs not needed)

**Publication notes:**

<!-- If documentation must be held or needs availability language, explain what reviewers should know and when it can be published. -->
### Public documentation

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.

Blocking: The docs writer's extraction step currently matches only Documentation and publication as the section heading. With this rename, PRs created from the template produce an empty PUBLICATION_READINESS, so generated docs PRs get the “did not provide publication-readiness information” fallback instead of the checked instruction. Please preserve the existing heading or update the consumer before this lands.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@buildsworth-bk I will address that in a separate PR.

<!-- The docs bot will act according to the following -->
- [ ] Ready for public documentation - document and publish
- [ ] Not ready for public docs - document and hold
- [ ] Not applicable - docs not needed

## Testing
- [ ] Tests have run locally (with `go test ./...`). Buildkite employees may check this if the pipeline has run automatically.
Expand Down
Loading