Michaël Zasso
Michaël Zasso
New run with the `install` `build` `test` sequence: https://github.com/targos/citgm/actions/runs/627579454
Should we skip on Node.js 10.x ?
Well, it failed on Node.js 10 because of ESM tests. I don't know why they aren't properly disabled but I think we can safely skip Node.js 10. It's in maintenance...
CI on all platforms: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker-pipeline/140/console
New CI: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker-pipeline/141/
@nodejs/citgm I think this is ready to land. There were two errors in the last CI run: https://ci.nodejs.org/job/citgm-smoker-nobuild/1050/nodes=win10-vs2019/console https://ci.nodejs.org/job/citgm-smoker-nobuild/1050/nodes=aix71-ppc64/console But they are not related to babel, as they happen before...
New CI: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker-pipeline/149/
`iconv` is handled in https://github.com/bnoordhuis/node-iconv/pull/194 and https://github.com/bnoordhuis/node-iconv/pull/189
`tape` fails because it apparently does not support to run on a node-pre version: ``` /home/iojs/build/workspace/citgm-smoker/nodes/debian8-64/citgm_tmp/83f5bbf7-0f6d-46d9-b25b-0c18ea8722f9/tape/node_modules/parse-node-version/index.js:6 throw new Error('Unable to parse: ' + version); ^ Error: Unable to parse: v12.0.0-pre...
I added sqlite3, leveldown and level to the list in https://github.com/nodejs/node/issues/25060