Community mailing list archives
Re: OCA and Merge proposalby
I always forget the review day happen on IRC, thank you o_o.
+33 (0) 6 70 74 52 26 | firstname.lastname@example.org
My blog : www.disruptingworld.me/
<img alt="Twitter" src="cid:email@example.com" height="20px"> <img alt="Github" src="cid:firstname.lastname@example.org" height="20px"> <img alt="Facebook" src="cid:email@example.com" height="20px"> <img alt="Google" src="cid:firstname.lastname@example.org" height="20px"> <img alt="RSS" src="cid:email@example.com" height="20px">
Building the tools we need
98 Avenue du general pierre billotte BatD
www.yolo-consulting.net | www.empower-us.net | www.wikicompare.info
Code reviews and functional reviews are all welcomed. So any skill is apreciated.
Hi Yannick. You can see all pull requests for OCA at https://github.com/pulls?user=OCA . Anyone can review. It is up to the committers to take the decision to merge at the end. It could in theory happen that a committer for some reason does not trust a review, but I've never been in such a situation so far. On Fri, Nov 21, 2014 at 2:52 PM, Yannick Buron <firstname.lastname@example.org> wrote: > Hello everyone, > > Some question regarding the OCA process : > > Do we have any place where we can see all merge proposals waiting review in > the OCA? Or do we have to go to each repo to see if a merge proposal is > pending? > > Also, who can be reviewer? I don't think being an OCA member is asked so > what is the skill required so someone can be considered skilled enough to > review a merge proposal? This is especially important for me since people > around me for Community verticalisation are still beginner in Odoo, and the > lack of reviewer may slow the project.