chore(repo): add CODEOWNERS for runtime ABI, CI, and release tooling#8617
chore(repo): add CODEOWNERS for runtime ABI, CI, and release tooling#8617jacekradko wants to merge 1 commit into
Conversation
🦋 Changeset detectedLatest commit: 7ebbd52 The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughAdds CODEOWNERS entries assigning ChangesHotloaded boundary ownership
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
Warning Review ran into problems🔥 ProblemsStopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a Comment |
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
There was a problem hiding this comment.
TIL I'm a member of "Team User SDK" https://github.com/orgs/clerk/teams/team-user-sdk
There was a problem hiding this comment.
TIL this team exists 😲
d77509f to
8aa6361
Compare
@clerk/team-sdk owns the hotload-boundary runtime surface that ships to consumer apps without version bumps (clerk-js core clerk.ts, react isomorphicClerk, ui contexts), plus the CI workflows, the CODEOWNERS policy itself, and the changeset config/changelog. Per-PR changeset entries are left unowned so they don't gate every PR. Empty changeset (repo-level config).
8aa6361 to
7ebbd52
Compare
Adds a
.github/CODEOWNERS(the repo has none today) covering the surfaces where an unreviewed change is most expensive, all owned by@clerk/team-sdk:clerk-js/src/core/clerk.ts,react/src/isomorphicClerk.ts,ui/src/contexts/), the surfaceAGENTS.mdcalls a contract;.github/workflows/, the CODEOWNERS file itself, and the changeset config/changelog.Per-PR changeset entries (
.changeset/*.md) are deliberately left unowned so they don't force a review on every PR.This only routes review requests until
require_code_owner_reviewsis enabled onmain; that branch-protection flip is a separate follow-up after this lands. Empty changeset since it's repo-level config.Summary by CodeRabbit