I’ve grouped the list a bit. Please take a look if that makes sense
(and feel free to use the list in guiding your reviews).
Obviously, this list is neither complete or authoritative, please help
to make it more useful (and not that “useful” != “longer” ;) ).
Cheers,
Till
On 9 Nov 2015, at 9:22, abdullah alamoudi wrote:
Hi Dev,
Just wanted to share with you that our code review checklist has been
uploaded to our wiki. Check it out at
https://cwiki.apache.org/confluence/display/ASTERIXDB/Code+Review+Checklist
, adhere to it and use it to make sure the code you're
submitting/reviewing
meets our minimum standard.
Also, feel free to contribute to the list, improve it, and if you can
automate some of the items in the list, that would be even better.
Cheers,
Abdullah.
