Skip to content
This repository has been archived by the owner on Aug 19, 2022. It is now read-only.

Update Infra #969 - Replace isparta with istanbul #1011

Merged
merged 1 commit into from
Dec 7, 2018

Conversation

blainekasten
Copy link
Contributor

Part of #969
Changes from using isparta to istanbul for test coverage.

Copy link
Member

@ryan-roemer ryan-roemer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to re-architect this a bit to produce a non-instrumented CommonJS lib/ to publish.

.babelrc Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@blainekasten blainekasten force-pushed the infra/istanbul-coverage branch 4 times, most recently from ed531e7 to 9e5b52e Compare December 5, 2018 20:17
.babelrc Outdated Show resolved Hide resolved
@blainekasten blainekasten force-pushed the infra/istanbul-coverage branch 2 times, most recently from 08f5571 to 9e5b52e Compare December 5, 2018 20:47
Copy link
Member

@ryan-roemer ryan-roemer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've pushed my changes fixing a pre-existing issue wherein we always instrumented + ran coverage. Now we only do it when running the coverage commands 😛

@blainekasten -- can you review my commits to make sure they're good and pull them down and kick the tires?

@kylecesmat -- can you give both of us a review?

thanks!

@blainekasten blainekasten force-pushed the infra/istanbul-coverage branch 3 times, most recently from 132232f to 262d629 Compare December 6, 2018 21:23
@blainekasten blainekasten merged commit c57556a into master Dec 7, 2018
@blainekasten blainekasten deleted the infra/istanbul-coverage branch December 7, 2018 05:35
@coveralls
Copy link

coveralls commented Dec 8, 2018

Coverage Status

Coverage increased (+0.5%) to 95.265% when pulling 132232f on infra/istanbul-coverage into 2d0c791 on master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants