Skip to content

chore: update CODEOWNERS for apps and deployments#8919

Merged
sriramveeraghanta merged 2 commits intopreviewfrom
chore/update-codeowners
Apr 20, 2026
Merged

chore: update CODEOWNERS for apps and deployments#8919
sriramveeraghanta merged 2 commits intopreviewfrom
chore/update-codeowners

Conversation

@sriramveeraghanta
Copy link
Copy Markdown
Member

@sriramveeraghanta sriramveeraghanta commented Apr 20, 2026

Summary

  • Assign per-app code owners: apps/api, apps/web, apps/space, apps/admin, apps/live
  • Keep existing owners for .oxlintrc.json, .oxfmtrc.json, and deployments/

Test plan

  • GitHub recognizes CODEOWNERS syntax (no parse warnings on the PR)
  • Reviews auto-request the listed owners on matching paths

Summary by CodeRabbit

  • Chores
    • Updated code ownership mappings for configuration files and application/deployment directories to streamline review routing and maintenance responsibilities.

Assign owners per app/area so reviews are routed to the right
maintainers.
Copilot AI review requested due to automatic review settings April 20, 2026 11:16
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2d9a575e-4a6d-4ce3-b9fa-928ad38ddfa6

📥 Commits

Reviewing files that changed from the base of the PR and between c7514cf and e715f3e.

📒 Files selected for processing (1)
  • CODEOWNERS
✅ Files skipped from review due to trivial changes (1)
  • CODEOWNERS

📝 Walkthrough

Walkthrough

Updated the CODEOWNERS file to add ownership entries for lint/format config files and to assign directory-level ownership for several app and deployment directories; the existing eslint.config.mjs ownership entry was left unchanged.

Changes

Cohort / File(s) Summary
CODEOWNERS
CODEOWNERS
Added ownership entries for config files .oxlintrc.json and .oxfmtrc.json, and directory-level ownership for apps/api/, apps/web/, apps/space/, apps/admin/, apps/live/, and deployments/. Existing eslint.config.mjs entry unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 I hopped through lines to mark each door,
Quietly naming who tends each floor.
Configs and apps now have a guide,
Paths assigned, no change to the tide.
A tidy map — I twitch with pride.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description provides a summary and test plan but is missing required sections from the template (Type of Change, Screenshots, detailed description). Complete the PR description by adding the Type of Change section and any other missing template sections for consistency with repository standards.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main change: updating CODEOWNERS for apps and deployments directories.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/update-codeowners

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates repository CODEOWNERS to reflect per-application ownership and deployment ownership rules so GitHub can auto-request reviews for changes in those areas.

Changes:

  • Add CODEOWNERS entries for .oxlintrc.json and .oxfmtrc.json
  • Add per-app ownership rules for apps/api, apps/web, apps/space, apps/admin, apps/live
  • Add ownership rule for deployments/ (and remove the old eslint.config.mjs rule)

Comment thread CODEOWNERS
Comment on lines +1 to +2
.oxlintrc.json @sriramveeraghanta @lifeiscontent
.oxfmtrc.json @sriramveeraghanta @lifeiscontent
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
CODEOWNERS (1)

1-8: Consider consistent whitespace formatting.

The spacing between path patterns and owner assignments appears inconsistent across lines. While this doesn't affect GitHub's parsing of the CODEOWNERS file, standardizing to consistent spacing (e.g., single space or aligned columns) would improve readability and maintainability.

📝 Optional formatting improvement
-.oxlintrc.json   `@sriramveeraghanta` `@lifeiscontent`
-.oxfmtrc.json   `@sriramveeraghanta` `@lifeiscontent`
-apps/api/           `@dheeru0198` `@pablohashescobar`
-apps/web/           `@sriramveeraghanta`
-apps/space/         `@sriramveeraghanta`
-apps/admin/         `@sriramveeraghanta`
-apps/live/          `@Palanikannan1437`
-deployments/        `@mguptahub` `@pratapalakshmi`
+.oxlintrc.json `@sriramveeraghanta` `@lifeiscontent`
+.oxfmtrc.json `@sriramveeraghanta` `@lifeiscontent`
+apps/api/ `@dheeru0198` `@pablohashescobar`
+apps/web/ `@sriramveeraghanta`
+apps/space/ `@sriramveeraghanta`
+apps/admin/ `@sriramveeraghanta`
+apps/live/ `@Palanikannan1437`
+deployments/ `@mguptahub` `@pratapalakshmi`
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@CODEOWNERS` around lines 1 - 8, Standardize spacing in the CODEOWNERS entries
so each path pattern (e.g., .oxlintrc.json, .oxfmtrc.json, apps/api/, apps/web/,
apps/space/, apps/admin/, apps/live/, deployments/) is followed by a consistent
delimiter spacing before the owner list; pick either a single space or aligned
columns and update all lines to match that convention to improve readability and
maintainability.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@CODEOWNERS`:
- Around line 1-8: Standardize spacing in the CODEOWNERS entries so each path
pattern (e.g., .oxlintrc.json, .oxfmtrc.json, apps/api/, apps/web/, apps/space/,
apps/admin/, apps/live/, deployments/) is followed by a consistent delimiter
spacing before the owner list; pick either a single space or aligned columns and
update all lines to match that convention to improve readability and
maintainability.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3a5715be-7b75-4182-b273-63885cb22e5b

📥 Commits

Reviewing files that changed from the base of the PR and between da41f14 and c7514cf.

📒 Files selected for processing (1)
  • CODEOWNERS

@sriramveeraghanta sriramveeraghanta merged commit 62b2d1b into preview Apr 20, 2026
9 checks passed
@sriramveeraghanta sriramveeraghanta deleted the chore/update-codeowners branch April 20, 2026 11:47
PhilippeCaira pushed a commit to PhilippeCaira/plane that referenced this pull request Apr 22, 2026
* chore: update CODEOWNERS for apps and deployments

Assign owners per app/area so reviews are routed to the right
maintainers.

* chore: update the codeowners
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants