On Tue, 2021-03-30 at 14:12 +0200, Ludovic Courtès wrote: > I respectfully think you misunderstand the review process. Review is > about sharing responsibilities and reducing the likelihood of > mistakes. > > It’s crucial from many different perspectives: security-wise, > socially > (the process is transparent, everyone gets a chance to chime in or to > just see what’s cooking), and technically (we all learn and build > better > software, and we reduce the risk that users receive broken software > on > their next ‘guix pull’). > > The process to get commit access is documented and it revolves around > two criteria: the person must have experience with the project and > must > know the rules. > > That’s not “bureaucracy”: it’s about building a community around > mutual > understanding of what each one of us may or may not expect from their > peers. > > > All that said, it’s of course fine to collaborate on external > repositories, even though I agree with Mark that getting “closer” to > Guix folks may be profitable to everyone. In the end, work should be > submitted for review, and for a patch set like this, it’ll probably > be a > ‘wip-’ branch on Savannah. > > Thanks, > Ludo’. I think Mark misrepresents my sayings in their messages. We are ONLY using git to exchange work and collaborate on the patchset with Raghav, how else do you want us to work with Raghav on some git branch on Savannah if Raghav doesnt have commit access? Don't we have the right with Raghav to cooperate together on some patchset however we want? It never meant that review process had to be skipped in ANY way. Only Raghav and myself has showed up to cooperate on GNOME 40 upgrade for now, and that's also why we are doing it this way. I feel bad now that it seems like everyone assumes we are doing things we are not. Léo