Skip to content

chore(codeowners): replace @swapnil-cometchat with @jitvarpatil - #504

Open
ketanyekale wants to merge 1 commit into
mainfrom
chore/codeowners-swapnil-to-jitvar
Open

chore(codeowners): replace @swapnil-cometchat with @jitvarpatil#504
ketanyekale wants to merge 1 commit into
mainfrom
chore/codeowners-swapnil-to-jitvar

Conversation

@ketanyekale

Copy link
Copy Markdown
Member

Replaces the repo-wide default owner entry, transferring it from @swapnil-cometchat to @jitvarpatil (who already owns /sdk/ and /ui-kit/). Single-line change; no other owner entries touched. Both handles verified to exist on GitHub.

-- @swapnil-cometchat
+- @jitvarpatil

⚠️ Note: this line is currently a no-op

The entry sits under a "Default owners for everything in the repo" comment, but the pattern is -, not *:

- @jitvarpatil

CODEOWNERS parses the first token as a gitignore-style file pattern. - matches only a file literally named -, so this rule almost certainly matches nothing today and there is effectively no repo-wide default owner. That is consistent with #458 having been merged without any review from the named default owner.

I deliberately did not change - to * in this PR, because that is a semantic change rather than the rename that was asked for: it would make @jitvarpatil a required reviewer on every PR touching an otherwise-unowned path. Happy to follow up with that fix if it is the intent — just say the word.

🤖 Generated with Claude Code

Transfers the repo-wide default owner entry to @jitvarpatil, who already
owns /sdk/ and /ui-kit/.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@mintlify

mintlify Bot commented Sep 9, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
cometchat 🟢 Ready View Preview Sep 9, 2026, 8:30 AM

💡 Tip: Enable Automations to automatically generate PRs for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants