Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run full CI only on node LTS #1692

Merged
merged 1 commit into from Jan 25, 2019
Merged

Conversation

IvanGoncharov
Copy link
Member

No description provided.

@IvanGoncharov IvanGoncharov changed the title Run full CI on node LTS Run full CI only on node LTS Jan 25, 2019
@codecov-io
Copy link

codecov-io commented Jan 25, 2019

Codecov Report

Merging #1692 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1692   +/-   ##
=======================================
  Coverage   97.77%   97.77%           
=======================================
  Files         214      214           
  Lines       13160    13160           
  Branches     1959     1959           
=======================================
  Hits        12867    12867           
  Misses        190      190           
  Partials      103      103

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d49460f...661979f. Read the comment docs.

@IvanGoncharov IvanGoncharov force-pushed the node11Tests branch 2 times, most recently from 5be96b3 to 8d0ac16 Compare January 25, 2019 15:06

jobs:
include:
- node_js: 'lts/*'
script: npm run test:ci
Copy link
Member Author

Choose a reason for hiding this comment

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

It duplicates tests on LTS but since all test jobs executed in parallel, it's not a big problem.

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

Successfully merging this pull request may close these issues.

None yet

3 participants