Skip to content
This repository was archived by the owner on Feb 6, 2018. It is now read-only.

adding code coverage#78

Merged
daffl merged 4 commits intomasterfrom
code-coverage
Oct 18, 2016
Merged

adding code coverage#78
daffl merged 4 commits intomasterfrom
code-coverage

Conversation

@ekryski
Copy link
Copy Markdown
Member

@ekryski ekryski commented Oct 17, 2016

It's not great on this repo

@ekryski ekryski changed the title adding code coverage. It's not great on this repo adding code coverage Oct 17, 2016
@ekryski
Copy link
Copy Markdown
Member Author

ekryski commented Oct 17, 2016

It's breaking on Node < 4 because of jsdom. @daffl what do you want to do here?

Drop jsdom entirely? Roll back to an older version? Or just build for node 4, 6 and latest. Personally I'd rather stick to our LTS and try and just pull jsdom. It was noisy with greenkeeper and we're only using it in one spot.

@ekryski
Copy link
Copy Markdown
Member Author

ekryski commented Oct 17, 2016

Actually it looks like feathers-socket-commons doesn't support node 0.10 and 0.12 so we may as well just drop those here as well.

The reason we didn't see that stuff is because we weren't running our tests against those versions on Travis.

@daffl
Copy link
Copy Markdown
Member

daffl commented Oct 18, 2016

Allright, I'm skipping the JSDom based jQuery tests now. We'll just add it to feathers-client.

@daffl
Copy link
Copy Markdown
Member

daffl commented Oct 18, 2016

Nevermind, you are right. Let's just remove the old Node version tests. We're still using Babel and don't plan any major changes here until the end of the year.

@daffl daffl merged commit 1d8bdd6 into master Oct 18, 2016
@daffl daffl removed the in progress label Oct 18, 2016
@ekryski ekryski deleted the code-coverage branch October 22, 2016 05:22
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.

2 participants