Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign uptools: remove unused global types from type-parser #21135
Conversation
nodejs-github-bot
added
doc
tools
labels
Jun 5, 2018
vsemozhetbyt
approved these changes
Jun 5, 2018
vsemozhetbyt
added
the
author ready
label
Jun 5, 2018
trivikr
approved these changes
Jun 5, 2018
This comment has been minimized.
This comment has been minimized.
|
Isn't it better to keep them for future usage? |
This comment has been minimized.
This comment has been minimized.
Maybe. Here are the counter-arguments, though:
|
targos
approved these changes
Jun 6, 2018
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Two CI failures are a known flaky and an infra problem Re-running. https://ci.nodejs.org/job/node-test-commit-linux/19394/ |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
arm-fanned re-run: https://ci.nodejs.org/job/node-test-commit-arm-fanned/1779/ |
This comment has been minimized.
This comment has been minimized.
|
arm-fanned again, we'll get to green sooner or later: https://ci.nodejs.org/job/node-test-commit-arm-fanned/1782/ |
This comment has been minimized.
This comment has been minimized.
|
Failing inspector tests that leave around stale processes that need to be manually terminated are absolutely brutal right now. They're basically making it impossible to get a green CI. Trying again anyway. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Landed in 6ac3c44 |
Trott commentedJun 5, 2018
tools/doc/type-parser.jshas several global types that are notused. Remove the unused global types.
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes