On Thursday, November 20, 2014 10:24:40 AM UTC+1, Pierre-Louis Bonicoli wrote: > > On 20/11/2014 01:42, Simon Klemenc wrote: > > The current contribution process seems very intransparent and it is > > not easy to see which party allowed for which change and who actually > > merged it. > > Hi, > > Could you please list the points that are missing/intransparent in the > related documentation : > https://code.google.com/p/tryton/wiki/HowtoContribute ? > > I would be happy to improve the documentation. > > -- > Pierre-Louis >
Hi pierre, sorry for putting this up top again, but from trying to contribute small pieces and looking at other codereviews a thing that remains unclear to me from howto contribute is pep8. how should this be handled? When doing modifications on a file with pep8 issues, should those issues be fixed with the patch? or with a seperate patchset? or should they be remained for the future? (making the reviewbot sort of pointless and more annoying than anything else) if the concensus is to have a very tightened workflow without too much space for communication, those things should be pointed out if it influences the result of a review... also i think the link from the vote results regarding the realname discussion is pointing to the wrong location (review 6451002) regards! simon
