cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vladimir Kotikov (Akvelon)" <v-vlk...@microsoft.com>
Subject RE: [DISCUSS] Enabling coverage reports collection and analysis for cordova repos
Date Thu, 04 Feb 2016 12:21:35 GMT
It looks like we don't need for INFRA here. The coverage reports is available without any integration
- see https://codecov.io/github/apache/cordova-lib?ref=39793cf6b7aaaf05eb9cfa03930d936c1cbf1f16
for the report.

-
Best regards, Vladimir

-----Original Message-----
From: Vladimir Kotikov (Akvelon) [mailto:v-vlkoti@microsoft.com] 
Sent: Thursday, February 4, 2016 2:49 PM
To: dev@cordova.apache.org
Subject: RE: [DISCUSS] Enabling coverage reports collection and analysis for cordova repos

Thank you, guys for the feedback.
This sounds like we now have a consensus, so I'm going to submit a ticket to INFRA, asking
to enable https://na01.safelinks.protection.outlook.com/?url=codecov.io&data=01%7c01%7cv-vlkoti%40064d.mgd.microsoft.com%7cdb618e5791e441c2c08b08d32d5926ce%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=6p36ovE7viGCUjyXnoRpSf1d9fvhNv1WuwRWU2RKFK0%3d
<-> GitHub integration, and send a PR to cordova-lib with coverage reports generation

-
Best regards, Vladimir

-----Original Message-----
From: Dmitry Blotsky [mailto:dblotsky@microsoft.com]
Sent: Thursday, February 4, 2016 7:49 AM
To: dev@cordova.apache.org
Subject: RE: [DISCUSS] Enabling coverage reports collection and analysis for cordova repos

Just an FYI for us: we do already have coverage reports: "npm run cover". This tool gives
extra functionality on top of that. I like the coverage diffs, and the GitHub reports. If
this is automated and free, I'm all for it. 

Kindly,
Dmitry

-----Original Message-----
From: Sergey Grebnov (Akvelon) [mailto:v-segreb@microsoft.com]
Sent: Tuesday, February 2, 2016 6:08 AM
To: dev@cordova.apache.org
Subject: RE: [DISCUSS] Enabling coverage reports collection and analysis for cordova repos

+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:csantana23@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-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%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: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%7c3ea9a18f25ee43194ac808d3
> > 2b
> > 5e79c2%7c72f988bf86f141af91ab2d7cd011db47%7c1&sdata=1SLJ4U2YHSQpiukr
> > Yi
> > 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%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-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
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org B KKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKCB
 [  X  ܚX KK[XZ[  ] ][  X  ܚX P ܙݘK \X K ܙ B  ܈Y][ۘ[  [X[  K[XZ[
 ] Z[ ܙݘK \X K ܙ B

---------------------------------------------------------------------
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
View raw message