+1 - I would start from identifying weak places - important functionality/methods which are not covered or coverage is low.
This is not just one magic coverage number - this is about what functionality is covered and what is not. -Sergey -----Original Message----- From: Carlos Santana [mailto:csantan...@gmail.com] Sent: Tuesday, February 2, 2016 4:49 PM To: dev@cordova.apache.org Subject: Re: [DISCUSS] Enabling coverage reports collection and analysis for cordova repos +1 This is cool tech to add to our project It shows more transparency which for me I'm all in !! It should make it easier when troubleshooting or fixing bugs, it goes one step forward from adding a test when a bug is found Also seeing how a PR affects coverage interesting info to see sometimes. Like Steve said don't go mad and kill your self looking trying reach 100% for the repo, use it in a way that make sense. On Tue, Feb 2, 2016 at 7:34 AM Vladimir Kotikov (Akvelon) < v-vlk...@microsoft.com> wrote: > This is not only about visualization. > Codecov also stores previous result and performs an analysis, how > coverage has changed. Take a look at the comment, made by codecov bot at the > PR: > https://github.com/vladimir-kotikov/cordova-lib/pull/2 > > - > Best regards, Vladimir > > -----Original Message----- > From: Dmitry Blotsky [mailto:dblot...@microsoft.com] > Sent: Tuesday, February 2, 2016 2:59 PM > To: dev@cordova.apache.org > Subject: Re: [DISCUSS] Enabling coverage reports collection and > analysis for cordova repos > > Is this basically just adding a UI then? > > Kindly, > Dmitry > > > On Feb 1, 2016, at 11:48 PM, Vladimir Kotikov (Akvelon) < > v-vlk...@microsoft.com> wrote: > > > >> What is the impact to our workflow? are we just adding a node > dependency, or are there other config/integration parts that we need to do? > > The only thing, required from our side is to replace `npm test` (or > > the > command, that executes jasmine tests) witn `npm run cover` in Travis > configuration (the coverage reports are already enabled for the most > important repos). However, some help from infra side is required to > setup Github <-> > https://na01.safelinks.protection.outlook.com/?url=codecov.io&data=01% > 7c01%7cdblotsky%40microsoft.com%7c92d8918c69a6441a775608d32ba557d4%7c7 > 2f988bf86f141af91ab2d7cd011db47%7c1&sdata=chjekNYEfx%2bpdSQBbcGVAYm5A3 > 8fPv%2fBfy1iDTyUau4%3d > integration. > > > >> You are using istanbul for code coverage it seems. > > Yup. > > > >> Why did you remove spec-cordova in your pr? Your [2] > > Only for demo purposes. I wanted to see how it will look like :) > > > >> I think it is a great idea to include this in our repos. It would > >> be > nice to track our coverage and work on improving it. > >> Though I do believe it is important not to chase 100% code coverage. > > Agree here, 100 % coverage is not the goal, We can just reach an > agreement on a minimal appropriate value and try to not to drop > coverage below this bar. > > > > - > > Best regards, Vladimir > > > > > > -----Original Message----- > > From: Jesse [mailto:purplecabb...@gmail.com] > > Sent: Tuesday, February 2, 2016 2:22 AM > > To: dev@cordova.apache.org > > Subject: Re: [DISCUSS] Enabling coverage reports collection and > > analysis for cordova repos > > > > This looks good, as a metric to add. > > What is the impact to our workflow? are we just adding a node > dependency, or are there other config/integration parts that we need to do? > > > > > > > > > > @purplecabbage > > https://na01.safelinks.protection.outlook.com/?url=risingj.com&data= > > 01 > > %7c01%7cv-vlkoti%40064d.mgd.microsoft.com%7c3ea9a18f25ee43194ac808d3 > > 2b > > 5e79c2%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=1SLJ4U2YHSQpiukr > > Yi > > pGlq7uBfl4uQNHxSE8q1JspDQ%3d > > > > On Mon, Feb 1, 2016 at 2:17 PM, Steven Gill <stevengil...@gmail.com> > wrote: > > > >> Hey Vlad! > >> > >> This looks great! You are using istanbul for code coverage it seems. > >> > >> Why did you remove spec-cordova in your pr? Your [2] > >> > >> I think it is a great idea to include this in our repos. It would > >> be nice to track our coverage and work on improving it. > >> > >> Though I do believe it is important not to chase 100% code coverage. > >> > >> > >> On Mon, Feb 1, 2016 at 6:21 AM, Vladimir Kotikov (Akvelon) < > >> v-vlk...@microsoft.com> wrote: > >> > >>> Hey, guys > >>> > >>> I've spent a couple of hours and have a prototype of how we can > >> visualize, > >>> track and analyze code coverage for our repos. > >>> Here is the fork of cordova-lib repo integrated with > >>> https://na01.safelinks.protection.outlook.com/?url=codecov.io&data > >>> =0 > >>> 1%7c01%7cv-vlkoti%40064d.mgd.microsoft.com%7c3ea9a18f25ee43194ac80 > >>> 8d > >>> 32b5e79c2%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=0oRS0Omv1Ba > >>> c3 OVYlTyT08ENLSP15s1%2fw%2fvXxcO39ZA%3d [1] (notice the coverage > >>> badge at the top of README), and a PR with some coverage metrics > >>> [2]. The coverage summary is placed here [3] > >>> > >>> If this proposal get accepted, I'd like to enable this feature for > >>> all > >> our > >>> repos, to encourage contributors to increase current test coverage > >>> up to appropriate values. > >>> > >>> It'd be great if anyone could give any feedback on this > >>> > >>> [1] https://github.com/vladimir-kotikov/cordova-lib > >>> [2] https://github.com/vladimir-kotikov/cordova-lib/pull/2 > >>> [3] > >>> https://na01.safelinks.protection.outlook.com/?url=https%3a%2f%2fc > >>> od > >>> ecov.io%2fgithub%2fvladimir-kotikov%2fcordova-lib&data=01%7c01%7cv > >>> -v > >>> lkoti%40064d.mgd.microsoft.com%7c3ea9a18f25ee43194ac808d32b5e79c2% > >>> 7c > >>> 72f988bf86f141af91ab2d7cd011db47%7c1&sdata=mtQlIC3U2lyzKA%2byFchr1 > >>> zl > >>> Yp9PuEUsFRH74kUaqLxg%3d > >>> > >>> - > >>> Best regards, Vladimir > >>> > >>> ------------------------------------------------------------------ > >>> -- > >>> - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > >>> For additional commands, e-mail: dev-h...@cordova.apache.org > >>> > >>> > >> > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > For additional commands, e-mail: dev-h...@cordova.apache.org > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > For additional commands, e-mail: dev-h...@cordova.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
