← Back to issue list

ci: add code owners

View original Github issue

Metadata

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

Current evaluation

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.

Resolution: closed resolved

Reason: PR snapcraft-rocks#114 was merged into main by maintainer mr-cal, approved by 2 reviewers with0 unresolved comments and no failing checks. Repo tooling had no pinned SHA, so the merge commit SHA could not be retrieved directly.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Adds code owners. (CRAFT-5209)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved 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.
stealth/space-bunny-alpha — closed resolved Merged CODEOWNERS configuration for CRAFT-5209 after two approvals; the nine-line, one-file change passed all listed CI checks.
qwen/qwen3.6-35b-a3b — — Merged after approval from two reviewers and passing all CI checks. The change adds code owners configuration to the repository, modifying one file with nine lines added. Addresses internal tracking ticket CRAFT-5209.
qwen3.6-35b-a3b-mtp-q6 — — Merged changes adding code owners to the CI configuration, resolving CRAFT-5209. Implemented directly by the maintainer without additional comments.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/craft-platforms#83 (confidence 70%)

    Same 'chore: add a CODEOWNERS file' change merged in craft-platforms as part of the broader CRAFT-5209 ownership rollout.

  • Related To: canonical/craft-application#589 (confidence 50%)

    Comparable CODEOWNERS PR in craft-application was closed unmerged, showing inconsistent adoption across Canonical craft projects.

Related issues

Issue Project State Summary Similarity
#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.
92%
#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.
90%
#567 ci: add code owners starbase merged PR #567 (CRAFT-5209) merged: adds a 12-line CODEOWNERS file to starbase to define repo code ownership and auto-assign reviewers; commit eeeb3db, all CI checks green.
89%
#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.
88%
#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.
87%
#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.
86%
#6296 ci: update code owners snapcraft merged 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.
84%
#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.
84%
#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.
84%
#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.
84%