Yeah I don't like that requirement at all. Sensible unit and integration test representation is a decent goal, but I don't like a code coverage req. On Fri, Dec 16, 2016 at 13:48 Michael Miklavcic <[email protected]> wrote:
> Can someone clarify this point in merge requirements? > > "All merged patches must have 100% test coverage." > > > On Fri, Dec 16, 2016 at 10:34 AM, James Sirota <[email protected]> wrote: > > > I incorporated the changes to the coding guidelines from our discuss > > thread. I'd like to get them voted on to make them official. > > > > > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=61332235 > > > > Please vote +1, -1, 0 > > > > The vote will be open for 72 hours. > > > > ------------------- > > Thank you, > > > > James Sirota > > PPMC- Apache Metron (Incubating) > > jsirota AT apache DOT org > > >
