Skip to content

chore(codeowners): drop perception ownership from Ivan - #4051

Merged
paul-nechifor merged 1 commit into
mainfrom
ivan/codeowners-perception
Sep 10, 2026
Merged

chore(codeowners): drop perception ownership from Ivan#4051
paul-nechifor merged 1 commit into
mainfrom
ivan/codeowners-perception

Conversation

@leshy

@leshy leshy commented Sep 10, 2026

Copy link
Copy Markdown
Member

Removing myself as the codeowner on /dimos/perception/.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CdVSkvTDVyeykV6i9uoAYh
@greptile-apps

greptile-apps Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

Safe to merge: perception files remain covered by the repository-wide ownership entry after the dedicated entry is removed.

What we checked:

  • Ran the general contract validation for the codeowners-global-fallback-check using the provided source. T-Rex
  • The current-output log showed PASS with exit code 0. T-Rex
  • Compared the before-output and after-output logs to verify the run's effect. T-Rex

Summary

  • Removes the dedicated ownership entry for dimos/perception/.
  • Perception files now use the existing repository-wide ownership entry, retaining codeowner coverage.
  • No issues found; this change is safe to merge.

Reviews (1) · Last reviewed commit: "chore(codeowners): drop perception owner..."

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #4051      +/-   ##
==========================================
- Coverage   78.22%   78.21%   -0.02%     
==========================================
  Files        1431     1428       -3     
  Lines      135204   135077     -127     
  Branches    11711    11708       -3     
==========================================
- Hits       105762   105645     -117     
+ Misses      26167    26111      -56     
- Partials     3275     3321      +46     
Components Coverage Δ
Tests 94.72% <ø> (+0.06%) ⬆️
Flag Coverage Δ
OS-ubuntu-24.04-arm 74.53% <ø> (-0.01%) ⬇️
OS-ubuntu-latest 75.10% <ø> (-0.01%) ⬇️
Py-3.10 75.09% <ø> (-0.01%) ⬇️
Py-3.11 75.10% <ø> (+<0.01%) ⬆️
Py-3.12 75.10% <ø> (ø)
Py-3.13 75.10% <ø> (ø)
Py-3.14 75.10% <ø> (ø)
Py-3.14t 75.10% <ø> (+<0.01%) ⬆️
SelfHosted-Large 30.77% <ø> (-0.03%) ⬇️
SelfHosted-Linux 35.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@leshy leshy changed the title chore(codeowners): drop perception ownership chore(codeowners): drop perception ownership from Ivan Sep 10, 2026
@paul-nechifor
paul-nechifor added this pull request to the merge queue Sep 10, 2026
@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Sep 10, 2026
Merged via the queue into main with commit 11f4958 Sep 10, 2026
48 of 52 checks passed
@paul-nechifor
paul-nechifor deleted the ivan/codeowners-perception branch September 10, 2026 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Required CI checks have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants