← Back to issue list

ci(github): add starcraft team as codeowners

View original Github issue

Metadata

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

Current evaluation

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.

Resolution: closed resolved

Reason: PR #425 was approved and merged into main as commit e07f06a ('ci(github): add starcraft team as codeowners'), adding a single line to .github/CODEOWNERS. The change was subsequently reverted by merged PR #437 (f9c9542) and later reintroduced by #747, #837 and #974.

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`? ----- Prerequisite to use self-hosted runners. Unblocks #330.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved 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.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR #425 to add the `starcraft` team as CODEOWNERS, enabling self-hosted runners and unblocking #330.

Update history

Date Change
created

Related work

  • Related To: canonical/craft-providers#330 (confidence 90%)

    PR description states this CODEOWNERS change unblocks #330 (buildd Ubuntu 23.10 support) as a prerequisite to using self-hosted runners.

  • Superseded By: canonical/craft-providers#437 (confidence 80%)

    Merged revert commit f9c9542 removed the starcraft teams from CODEOWNERS to stop automatic review requests, reversing #425.

  • Related To: canonical/craft-providers#747 (confidence 70%)

    Commit e68ff2c 'chore: add CODEOWNERS' re-added the starcraft code owner entries after the #437 revert.

Related issues

Issue Project State Summary Similarity
#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.
85%
#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.
78%
#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%
#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.
78%
#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).
77%
#995 chore: update codeowners rockcraft merged Merged as commit 7157c17c, adding starcraft-authors to CODEOWNERS and assigning @steinbro as the group owner.
74%
#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.
74%
#226 ci: add code owners craft-archives merged PR #226 by maintainer mr-cal merged (commit c566fce, CRAFT-5209), adding a 13-line .github/CODEOWNERS: @lengau as global owner, starcraft-authors/reviewers for docs, maintainers required for CODEOWNERS edits.
73%
#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.
71%
#385 ci: add code owners craft-store merged Merged into main via commit 6fdd41e: adds a 13-line .github/CODEOWNERS (global owner @bepri, docs and maintainer ownership rules) for CRAFT-5209, tracked by canonical/starflow#154.
70%