We have these already written down in from both the reviewing and development perspective: http://cassandra.apache.org/doc/latest/development/how_to_review.html http://cassandra.apache.org/doc/latest/development/testing.html
My takeaway from recent in-person discussions the other week about this topic, is that it's really just a matter of making this less of a chore/sharing the burden of reviews when the whole dtest suite has to be run. Thanks for bringing this up, though. On Tue, Sep 20, 2016 at 6:42 AM, sankalp kohli <kohlisank...@gmail.com> wrote: > Hi, > I wanted to know if there are any guidelines for contributors to give > out unit and integration tests along with the patches. If not, we should > discuss and have them in place. > > I know we are making good progress with test coverage but we should add > guidelines around adding unit and integration tests with every patch. > Thoughts on this? > > Thanks, > Sankalp > -- ----------------- Nate McCall Wellington, NZ @zznate CTO Apache Cassandra Consulting http://www.thelastpickle.com
