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
test: test on travis with node 10 #2461
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2461 +/- ##
=======================================
Coverage 99.63% 99.63%
=======================================
Files 29 29
Lines 822 822
=======================================
Hits 819 819
Misses 3 3 Continue to review full report at Codecov.
|
@@ -24,6 +24,7 @@ describe('test/doc.test.js', () => { | |||
|
|||
it('should no broken url', async () => { | |||
const result = await findlinks({ src: app.url, logger: console }); | |||
if (result.fail !== 0) console.log(result); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@popomore 没有打出具体是哪个 broken 了
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好像有 bug
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
等 doctools 修复了再合并吧
.travis.yml
Outdated
@@ -3,6 +3,7 @@ language: node_js | |||
node_js: | |||
- '8' | |||
- '9' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
去掉 9 的
rebase 一下 |
@popomore 有空看看 doctools 的报错吧 |
👍 |
test: test on travis with node 10 (#2461)
Checklist
npm test
passesAffected core subsystem(s)
Description of change