← Back to issue list

chore(codeowners): remove authors from changelog reviews

View original Github issue

Metadata

Project
craft-parts
Number
#1461
Type
pull request
State
merged
Author
bepri
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #1461 (bepri) fixing the .github/CODEOWNERS template: changelog rules now target changelog.rst instead of release-notes.rst, so library changelog edits no longer ping TAs. 1-line diff, all CI green.

Resolution: closed resolved

Reason: The PR itself was merged into main after approval by 2 reviewers with all CI checks passing. The single-line change to the .github/CODEOWNERS template replaces the release-notes.rst path with changelog.rst, matching the libraries' actual changelog filename.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? --- The CODEOWNERS template file assumes that a given repository is using `release-notes.rst`, but the libraries generally use `changelog.rst` instead. This updates the file to behave like it should and not ping the TAs unnecessarily for library changelog updates.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1461 (bepri) fixing the .github/CODEOWNERS template: changelog rules now target changelog.rst instead of release-notes.rst, so library changelog edits no longer ping TAs. 1-line diff, all CI green.
stealth/space-bunny-alpha — closed resolved Merged a one-line CODEOWNERS update from release-notes.rst to changelog.rst, preventing unnecessary reviewer notifications for library changelog updates.
stealth/space-bunny-alpha — closed resolved Merged canonical/craft-parts#1461 (f8ce2833), correcting the CODEOWNERS template to reference changelog.rst and avoid unnecessary TA notifications for library changelog updates.
stealth/space-bunny-alpha — closed resolved Merged a CODEOWNERS fix to use changelog.rst instead of release-notes.rst, preventing unnecessary review notifications for library changelog updates.
stealth/space-bunny-alpha — closed resolved Merged craft-parts#1461, updating the CODEOWNERS template to target changelog.rst instead of release-notes.rst so library changelog updates no longer notify TAs unnecessarily.
qwen/qwen3.6-35b-a3b — — Merged a CODEOWNERS template update to correctly reference changelog.rst instead of release-notes.rst, preventing unnecessary TA notifications for library updates. Approved by two reviewers with all CI checks passing.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix to the CODEOWNERS template that replaces the incorrect release-notes.rst reference with changelog.rst. This prevents unnecessary notifications to TAs when library changelogs are updated.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#392 chore: remove codeowners starbase merged Merged commit 3b0bad9 removed the 9-line .github/CODEOWNERS file as agreed in standup; the PR was approved twice and all CI checks passed.
81%
#2647 chore: update `CODEOWNERS` charmcraft merged Merged into main as fa075d8c, updating .github/CODEOWNERS with four ownership-line changes.
78%
#429 chore: update codeowners craft-cli merged Merged current-year CODEOWNERS updates via commit 5ae642f; PR #429 was approved and all CI checks passed.
76%
#177 chore: update codeowners owner craft-platforms merged Merged PR craft-platforms#177 (1 line changed in .github/CODEOWNERS): docs/ ownership added to @canonical/starcraft-authors, changelog to starcraft-reviewers; now present in the repo at main.
76%
#837 chore: update CODEOWNERS craft-providers merged Merged PR #837 updated project CODEOWNERS assignments; the change landed as commit a0967cc.
76%
#62 ci: add code owners craft-actions merged PR #62 merged as commit b673d3d, adding a 9-line .github/CODEOWNERS (global owner @bepri, maintainer approval required for CODEOWNERS edits) for CRAFT-5209. Approved by 2 reviewers, all CI checks passed.
75%
#747 chore: add CODEOWNERS craft-providers merged PR #747 was merged into main via commit e68ff2c, adding .github/CODEOWNERS to define global, documentation, and ownership-rule review responsibilities.
75%
#937 chore: update CODEOWNERS craft-application merged Merged chore PR #937 (commit 3d31a9fa) updating .github/CODEOWNERS: changelog.rst ownership moved to @canonical/starcraft-reviewers, other docs to starcraft-authors, and CODEOWNERS itself to starcraft-maintainers.
75%
#2035 chore: add CODEOWNERS file charmcraft merged Merged PR #2035 (commit af590552) by maintainer lengau after 2 approvals, adding a 12-line .github/CODEOWNERS file setting @lengau as global owner plus docs/templates ownership rules.
75%
#179 chore: remove authors from changelog codeowners craft-platforms merged Merged PR #179 via commit 4605b38, moving general docs ownership to @canonical/starcraft-authors and assigning the changelog to @canonical/starcraft-reviewers.
74%