← Back to issue list

ci: update code owners

View original Github issue

Metadata

Project
snapcraft
Number
#6296
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

PR #6296 by maintainer mr-cal merged as commit c5073fffa, updating .github/CODEOWNERS with six ownership entry changes for CRAFT-5209. Approved by 2 reviewers; core CI passed, only unrelated integration-spread and OSV-scanner jobs failed.

Resolution: closed resolved

Reason: The pull request itself was the change: it was merged into main as commit c5073fffa ("ci: update code owners (#6296)"), touching .github/CODEOWNERS (+6/-6 lines). Failing integration-spread-tests and OSV-scanner checks are unrelated to this metadata-only change.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Updates code owners. (CRAFT-5209) --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [ ] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #6296 by maintainer mr-cal merged as commit c5073fffa, updating .github/CODEOWNERS with six ownership entry changes for CRAFT-5209. Approved by 2 reviewers; core CI passed, only unrelated integration-spread and OSV-scanner jobs failed.
stealth/space-bunny-alpha — closed resolved Merged PR #6296 (commit c5073fffa), updating CODEOWNERS with six ownership changes for CRAFT-5209; core CI checks passed despite unrelated integration/OSV failures.
qwen/qwen3.6-35b-a3b — — Merged following approval by two reviewers. Updates code owners configuration with a six-line change. Core CI checks including linting, unit tests, and publishing passed successfully.
qwen3.6-35b-a3b-mtp-q6 — — Merged maintainer mr-cal's pull request updating the repository's code owners configuration to reflect current team responsibilities, addressing CRAFT-5209. Integrated into main without additional comments.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#5967 (confidence 60%)

    Earlier CODEOWNERS maintenance commit (7e702a7cb, 'chore(codeowners): bring @mr-cal back') touching the same file.

  • Related To: canonical/snapcraft#5765 (confidence 50%)

    Prior 'ci: update codeowners' commit (b65275b66) in the same recurring maintenance series.

  • Related To: canonical/snapcraft#5177 (confidence 40%)

    Original commit adding the CODEOWNERS file (fc8cfbb26).

Related issues

Issue Project State Summary Similarity
#1095 ci: update code owners craft-application merged Merged PR #1095 (commit 01ee7b25, "ci: update CODEOWNERS") by mr-cal, rewriting .github/CODEOWNERS (+6/-7) to reassign code ownership per CRAFT-5209; approved by 2 reviewers despite unrelated snap-tests/OSV-scanner CI failures.
91%
#454 ci: update code owners craft-cli merged PR #454 merged as commit 60b6511 ("ci: update code owners"), adjusting .github/CODEOWNERS ownership entries (CRAFT-5209); +6/-8 in one file, approved by 2 reviewers with all CI checks green.
90%
#974 ci: update code owners craft-providers merged Routine maintenance PR updating .github/CODEOWNERS (CRAFT-5209); approved by 2 reviewers and merged as commit 1543896 despite unrelated pre-existing CI failures.
89%
#1260 ci: update code owners rockcraft merged Merged PR #1260 refreshed .github/CODEOWNERS in commit 574af710, applying the approved CRAFT-5209 ownership update.
89%
#235 ci: update code owners craft-platforms merged Merged PR canonical/craft-platforms#235 by maintainer mr-cal, landing CRAFT-5209 CODEOWNERS updates as commit 593f3ef ("ci: update code owners (#235)"). Approved by 2 reviewers; all CI checks passed.
89%
#358 ci: add code owners imagecraft merged Merged PR #358 by maintainer mr-cal adds a 12-line .github/CODEOWNERS file to imagecraft to define code ownership (CRAFT-5209); approved by 2 reviewers, all CI checks passing.
89%
#167 ci: add code owners debcraft merged PR #167 by maintainer mr-cal was approved (2 reviewers) and merged as commit a2c4077, adding .github/CODEOWNERS (CRAFT-5209) with global, docs, and self-approval owners. The only failing check was Security scan / OSV-scanner, which is unrelated to this 12-line config addition.
88%
#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.
86%
#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.
85%
#114 ci: add code owners snapcraft-rocks merged Merged PR adding a 9-line CODEOWNERS file (+9/-0, 1 file) for CRAFT-5209; approved by two reviewers with all listed CI checks passing.
84%