Hi ! While discussing with Cédric [1], I discovered that the workflow which describes how contributions are accepted was not completely documented.
This is important to describe this workflow: new contributor need it [2], so documenting it allows us to focus on the code. I believed - it was my understanding of [3] and [4] - that any patch to framework sub-project (the Tryton server and client) must be approved by a "LGTM" of Cédric. It occurs that it was a misunderstanding [2] :) The rules - as explained by Cédric [1], are: Any core developer is allowed to push, core developers take responsibility. The core developers are: Cédric, nicoe, pokoli and albert. People allowed to push without being core developers are allowed to push small fixes without LGTM. Bigger fixes need approval (LGTM) of a core developer. For bigger fixes, in case of disagreement a consensus should be reached, at last the project leader takes a decision. If nobody disagrees, I will update [4] in one week (if you miss this dead line, discussion could take place later :) [1] French: http://www.tryton.org/~irclog/fr/2014-08-20.log.html#t2014-08-20%2011:35 [2] http://www.tryton.org/~irclog/2014-01-31.log.html#t2014-01-31%2012:47 [3] https://code.google.com/p/tryton/wiki/ProjectOrganization/ [4] https://code.google.com/p/tryton/wiki/HowtoContribute -- Pierre-Louis
