cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Carlos Santana <csantan...@gmail.com>
Subject Re: [DISCUSS] Enabling coverage reports collection and analysis for cordova repos
Date Tue, 02 Feb 2016 13:49:17 GMT
+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-vlkoti@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:dblotsky@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-vlkoti@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%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=chjekNYEfx%2bpdSQBbcGVAYm5A38fPv%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:purplecabbage@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%7c3ea9a18f25ee43194ac808d32b
> > 5e79c2%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=1SLJ4U2YHSQpiukrYi
> > pGlq7uBfl4uQNHxSE8q1JspDQ%3d
> >
> > On Mon, Feb 1, 2016 at 2:17 PM, Steven Gill <stevengill97@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-vlkoti@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%7c3ea9a18f25ee43194ac808d
> >>> 32b5e79c2%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=0oRS0Omv1Bac3
> >>> 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%2fcod
> >>> ecov.io%2fgithub%2fvladimir-kotikov%2fcordova-lib&data=01%7c01%7cv-v
> >>> lkoti%40064d.mgd.microsoft.com%7c3ea9a18f25ee43194ac808d32b5e79c2%7c
> >>> 72f988bf86f141af91ab2d7cd011db47%7c1&sdata=mtQlIC3U2lyzKA%2byFchr1zl
> >>> Yp9PuEUsFRH74kUaqLxg%3d
> >>>
> >>> -
> >>> Best regards, Vladimir
> >>>
> >>> --------------------------------------------------------------------
> >>> - To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
> >>> For additional commands, e-mail: dev-help@cordova.apache.org
> >>>
> >>>
> >>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
> For additional commands, e-mail: dev-help@cordova.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
> For additional commands, e-mail: dev-help@cordova.apache.org
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message