Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Setup codeowners file with new team structure #1864

Merged
merged 4 commits into from
Oct 22, 2024
Merged

Setup codeowners file with new team structure #1864

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
fb93432
setting up codeowners for o1js
45930 Oct 14, 2024
e0e0599
update ownership to always include admins
45930 Oct 14, 2024
1a56b0a
updating again
45930 Oct 15, 2024
4e1f855
simplify codeowners to bare bones
45930 Oct 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion CODEOWNERS
View file
Open in desktop
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1 +1,7 @@
/src/lib/gadgets @o1-labs/crypto-eng-reviewers @mitschabaude
# By default, review is required by o1js-admins
* @o1-labs/o1js-admins

# Changes to the following paths should be reviewed by o1js-admins, but we will tag current and future zkapps teams as well (incorrect use of codeowners)
# We are relying on good faith to wait for admin approval on these PRs
# Dangerous PRs that could release a new version, new consensus rules, or anything like that will necessarily touch more than `/src/lib`, so this is minimally risky
/src/lib @o1-labs/o1js-admins @o1-labs/exploration-team @o1-labs/future-zkapps
Loading