Skip to content

docs: keep generation footers out of pull requests and commits - #42

Merged
N4M3Z merged 1 commit into
mainfrom
change/no-generation-footer
Aug 25, 2026
Merged

docs: keep generation footers out of pull requests and commits#42
N4M3Z merged 1 commit into
mainfrom
change/no-generation-footer

Conversation

@N4M3Z

@N4M3Z N4M3Z commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

A pull request body and a commit message carry no generation footer, no tool badge, and no session link. Attribution lives in the author line and the authors.yaml trailers. The change edits both CONTRIBUTING copies.

ignore:spec: the change edits contributor prose and no check.

Release Notes

  • Contributor guidance excludes generation footers, tool badges, and session links from pull request bodies and commit messages.

A pull request body and a commit message carry no generation
footer, no tool badge, and no session link. Attribution lives in
the author line and the authors.yaml trailers. Apply the rule to
both CONTRIBUTING copies.

ignore:spec: the change edits contributor prose and no check.

Co-Authored-By: Martin Zeman <N4M3Z@users.noreply.github.com>
@N4M3Z N4M3Z added the ignore:spec Allowed defect: protected paths change without a specification, reason in body label Aug 24, 2026
@github-actions github-actions Bot added size:small auto: changed-lines bucket area:docs auto: documentation and markdown area:templates and removed size:small auto: changed-lines bucket labels Aug 24, 2026
@N4M3Z

N4M3Z commented Aug 24, 2026

Copy link
Copy Markdown
Contributor Author

@cursor review

@cursor

cursor Bot commented Aug 24, 2026

Copy link
Copy Markdown

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

@runewright runewright Bot added the issue:cursor Provider blocker: clear the cursor problem before re-summoning label Aug 24, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 24, 2026

Copy link
Copy Markdown

Approvability

Verdict: Approved at ec6618c

Macroscope's review found this PR approvable — This is a documentation-only update that records and propagates guidance to omit generation footers from pull requests and commits, without changing executable or production behavior. All modified files are owned by the author, and the scope is small and self-contained.

Notes:

  • Macroscope's correctness review did not run, so approvability was decided on eligibility alone.

You can add or adjust custom eligibility rules. Learn more.

@N4M3Z

N4M3Z commented Aug 24, 2026

Copy link
Copy Markdown
Contributor Author

Failing step: Quality, Release notes attestation.

Cause: The pull request body has no Release Notes section with a list entry.

Suggested fix: Add a Release Notes heading and one list entry. Use - N/A for non-user-facing work.

@github-actions github-actions Bot added the size:small auto: changed-lines bucket label Aug 24, 2026
@N4M3Z N4M3Z added the review:runeseer Summons the adjudicating correctness lane label Aug 24, 2026
@runeseer

runeseer Bot commented Aug 24, 2026

Copy link
Copy Markdown

Runeseer review — clean

Looks good. The change edits contributor prose in both CONTRIBUTING copies plus a CHANGELOG entry, and authors.yaml exists at the repo root and under templates/base/, so the attribution claim both copies make holds.

  • No inline findings from the other reviewers; the automated approval covered eligibility only.
  • Minor: the new ### Changed block sits above ### Added under [Unreleased], the reverse of the Keep a Changelog order used elsewhere.

No open findings · Reviewed ec6618c5 · review run · $0.37 · 0m36s · 11 turns

@runeseer runeseer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Earned approval: review/correctness returned a clean verdict for ec6618c. Dismissed automatically on any new push.

@github-actions github-actions Bot removed the review:runeseer Summons the adjudicating correctness lane label Aug 24, 2026
@N4M3Z
N4M3Z merged commit 4312b94 into main Aug 25, 2026
43 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:docs auto: documentation and markdown area:templates ignore:spec Allowed defect: protected paths change without a specification, reason in body issue:cursor Provider blocker: clear the cursor problem before re-summoning size:small auto: changed-lines bucket

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant