Thanks everyone, that was a fantastic meeting. Thi...
# contributea
average-finland-92144
03/30/2023, 8:24 PMThanks everyone, that was a fantastic meeting. This is quickly becoming my favorite meeting of the week!
Recording will be posted in the YT channel.
There were a couple of housekeeping items that we didn't have the chance to discuss there:
🔥 3
average-finland-92144
03/30/2023, 8:29 PM1. Branches vs forks. For contributing to Flyte, do we want to require PRs to come from forks exclusively? or keep the current approach of creating branches in the repo. @high-accountant-32689 @freezing-airport-6809. The decision could have implications on the privileges and requirements for new Contributors
2. For the CODEOWNERS idea from @cool-lifeguard-49380. Is it the goal of it to auto assign TSC members (at least) as reviewers for new RFCs?
h
hallowed-mouse-14616
03/30/2023, 8:41 PM1. I think right now we allow Flyte maintainers (or contributors?) to open branches in the Flyte repos. The general idea is that users must fork for the first few contributions and then they'll be added to the requisite permissions to allow branching directly. Not very opinionated on this moving forward, but just some context.
thx 1
h
high-accountant-32689
03/31/2023, 12:57 AM1. +1 to what Dan said. Maintainers have permissions to open branches in the repo. For all other contributions we follow the usual OSS process of forking the repo.
2. This is reasonable. This will solve the problem of awareness for all new RFCs.
c
cool-lifeguard-49380
03/31/2023, 7:31 AM1. I’m personally also just fine with having a fork and creating PRs from there, this is what I’ve done so far. However you prefer.
cool-lifeguard-49380
03/31/2023, 7:31 AM2. Yes, my idea was automatically becoming aware of RFCs to review
a
average-finland-92144
03/31/2023, 3:56 PMthanks everyone
so regarding #1, and to comply with the RFC process, I'll submit a short RFC to change the current privileges of the Contributor role (which include branching) to make it forks only. Branching would remain a privilege for maintainers.
There are concerns, though, that this movement could elevate the barriers for someone to become a contributor. I'll let elaborate @thankful-minister-83577 better on this as a comment in the PR to come
152 Views