Skip to content

docs(codeowners): add profiler co-owners for assets/profiler-heldout.js - #824

Merged
yakew7 merged 1 commit into
yakew7:mainfrom
Tiyatrotist:fix/codeowners-profiler-heldout-823
Oct 6, 2026
Merged

yakew7 merged 1 commit into
yakew7:mainfrom
Tiyatrotist:fix/codeowners-profiler-heldout-823

Conversation

@Tiyatrotist

Copy link
Copy Markdown
Contributor

Summary

Adds /assets/profiler-heldout.js @yakew7 @ahmdkaml to .github/CODEOWNERS so review routing for profiler changes includes profiler co-owners, and adds a test in tests/test_codeowners.py guarding that profiler scripts are co-owned.

Type

  • Audit
  • Explainer
  • Bug fix
  • Other

Linked issue

Closes #823

@vercel

vercel Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

@Tiyatrotist is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

@yakew7 @ahmdkaml - new PR touching a path you own, please review.

@yakew7
yakew7 merged commit 8034835 into yakew7:main Oct 6, 2026
18 of 19 checks passed
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.

assets/profiler-heldout.js is missing from CODEOWNERS' profiler co-owner entries

2 participants