← Back to issue list

ci(github): remove starcraft team as codeowners

View original Github issue

Metadata

Project
craft-providers
Number
#437
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #437 (commit f9c9542) reverted #425 and removed the Starcraft team from CODEOWNERS, stopping unwanted automatic review requests.

Resolution: closed resolved

Reason: PR #437 was approved and merged as commit f9c9542, explicitly reverting PR #425. The change removed the Starcraft CODEOWNERS entry to prevent automatic review requests for work not ready for review.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? ----- This reverts https://github.com/canonical/craft-providers/pull/425. The codeowners feature is a nuisance - it automatically sends review requests, even when the author is not ready for a PR review.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #437 (commit f9c9542) reverted #425 and removed the Starcraft team from CODEOWNERS, stopping unwanted automatic review requests.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR #437 to revert #425, removing the starcraft team from CODEOWNERS to prevent automatic review requests.

Update history

Date Change
created

Related work

Related issues

Issue Project State Summary Similarity
#425 ci(github): add starcraft team as codeowners craft-providers merged Merged as commit e07f06a, adding @canonical/starcraft teams to .github/CODEOWNERS as a prerequisite for self-hosted runners; it unblocked #330. Later reverted by #437 (f9c9542) and re-added in #747.
85%
#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).
72%
#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.
72%
#622 ci: add code owners fetch-service merged Merged PR #622 as commit 60dbee3 ("ci: add code owners"), adding a 12-line .github/CODEOWNERS (CRAFT-5209) with @cmatsuoka as global owner, starcraft-authors on /docs, and starcraft-maintainers required for CODEOWNERS edits.
71%
#995 chore: update codeowners rockcraft merged Merged as commit 7157c17c, adding starcraft-authors to CODEOWNERS and assigning @steinbro as the group owner.
70%
#57 feat(renovate): add starcraft-reviewers as default reviewers starflow merged Merged canonical/starflow#57 as commit 23d57bb, adding starcraft-reviewers as a default reviewer team for all Renovate pull requests.
70%