← Back to issue list

chore: remove authors from changelog codeowners

View original Github issue

Metadata

Project
craft-platforms
Number
#179
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #179 via commit 4605b38, moving general docs ownership to @canonical/starcraft-authors and assigning the changelog to @canonical/starcraft-reviewers.

Resolution: closed resolved

Reason: PR #179 was merged into the main branch in commit 4605b38. It updated CODEOWNERS so changelog reviews are handled by the reviewers team, making author review opt-in.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Having the authors own the changelog is a heavy burden for them in our libraries that's minimally useful. Per @medubelko, from now on, TA reviews for the changelog are opt-in when we decide we need their input. Drive-by: changes docs ownership in general to the authors group - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint && make test`? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #179 via commit 4605b38, moving general docs ownership to @canonical/starcraft-authors and assigning the changelog to @canonical/starcraft-reviewers.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR removing the starcraft-authors team from CODEOWNERS for the changelog, reducing maintainer burden and making TA reviews opt-in.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#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.
84%
#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.
84%
#1361 chore: make the starcraft-reviewers team own the changelog craft-parts closed PR to move docs/reference/changelog.rst ownership to starcraft-reviewers was closed unmerged as superseded by #1371 (also closed unmerged); the change landed via merged #1372 (commit 3d369498).
81%
#995 chore: update codeowners rockcraft merged Merged as commit 7157c17c, adding starcraft-authors to CODEOWNERS and assigning @steinbro as the group owner.
81%
#1356 chore: update codeowners craft-parts merged Merged PR #1356 (commit 91f02a35) updated .github/CODEOWNERS: added the starcraft-authors group and set @steinbro as the CODEOWNERS owner (+2/-2).
79%
#295 chore: add CODEOWNERS file starbase merged Merged PR canonical/starbase#295 (commit 913e475) by maintainer lengau added a 9-line .github/CODEOWNERS with @bepri as global owner and starcraft-authors for /docs/. Later removed in 3b0bad9 and re-added in #567.
78%
#1461 chore(codeowners): remove authors from changelog reviews craft-parts merged 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.
74%
#83 chore: add a CODEOWNERS file craft-platforms merged PR #83 merged as commit 1da9c16, adding .github/CODEOWNERS (+20 lines) to define global, docs, per-module (charm/rock/snap) and starcraft-maintainer review ownership; approved by 2 reviewers.
73%
#154 ci: add code owners starflow merged PR merged by maintainer mr-cal (CRAFT-5209), adding a .github/CODEOWNERS file (12 lines) assigning @lengau as global owner, @canonical/starcraft-authors on /docs/, and @canonical/starcraft-maintainers for CODEOWNERS changes.
72%
#437 ci(github): remove starcraft team as codeowners craft-providers merged Merged PR #437 (commit f9c9542) reverted #425 and removed the Starcraft team from CODEOWNERS, stopping unwanted automatic review requests.
72%