Skip to content

docs: add pull request template with cross-repo checklist - #1034

Open
mbevc1 wants to merge 1 commit into
mainfrom
20260726_pr_template
Open

docs: add pull request template with cross-repo checklist#1034
mbevc1 wants to merge 1 commit into
mainfrom
20260726_pr_template

Conversation

@mbevc1

@mbevc1 mbevc1 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Adds pull_request_template.md so every PR gets a description prompt and a checklist reminding authors to check related surfaces: Helm chart (separate PR), Terraform provider, and docs.

@mbevc1 mbevc1 added the documentation Improvements or additions to documentation label Jul 26, 2026
@claude

claude Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Claude finished @mbevc1's task in 42s —— View job


Review: PR #1034 — PR template with cross-repo checklist

  • Read the changed file (.github/pull_request_template.md)
  • Validate cross-repo / path references
  • Post review feedback

Summary: Clean, well-scoped docs-only change. This adds a single .github/pull_request_template.md with a description prompt and a checklist covering the surfaces that tend to be forgotten (Helm chart, Terraform provider, autogenerated docs). No Go code changed, so go build / go vet / go mod tidy don't apply here.

Verified

  • ✅ The charts/k8s-reporter/ path referenced on line 7 exists in the repo.
  • ✅ The grammar concern from the previous review is resolved — line 7 now reads cleanly (checked / updated, if needed. Note: this lives in a **separate PR**).

Nice work — the "check related surfaces" checklist is a thoughtful addition that should reduce cross-repo drift, and calling out that the Helm chart lives in a separate PR pre-empts a common point of confusion. 👍

Nothing blocking here.
20260726_pr_template

Comment thread .github/pull_request_template.md Outdated
Adds `pull_request_template.md` so every PR gets a description
prompt and a checklist reminding authors to check related surfaces:
Helm chart (separate PR), Terraform provider, and docs.
@mbevc1
mbevc1 force-pushed the 20260726_pr_template branch from bc2d876 to b673537 Compare July 26, 2026 21:21
@mbevc1
mbevc1 enabled auto-merge (squash) July 26, 2026 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant