Ecosystem Modules that will break in Node 12

[targos]

This is an issue meant to track modules that are currently skipped in citgm because they (or their tests) are currently broken on current versions of Node.

Fixed	Module name	Node version	Reason	Refs
	uglify-js	master	Output expectation in tests	mishoo/UglifyJS#3304
	time	11, master	Crash when called with NaN	TooTallNate/node-time#76
	stylus	11, master	Fixed version unreleased	stylus/stylus#2436
	graceful-fs	11, master	Tests broken	isaacs/node-graceful-fs#137
	bcrypt	master	Using removed V8 API	kelektiv/node.bcrypt.js#684
	microtime	master	Using removed V8 API	wadey/node-microtime#61
	serialport	master	Using removed V8 API	serialport/node-serialport#1742
	ref	master	Using removed V8 API	TooTallNate/ref#107
	node-report	master	Using removed V8 API	nodejs/node-report#116
	sqlite3	master	Using removed V8 API	TryGhost/node-sqlite3#1093
	leveldown	master	Using removed V8 API	Level/leveldown#540
	level	master	Using removed V8 API	Level/level#123
	iconv	master	Using removed V8 API	bnoordhuis/node-iconv#198
	node-gyp	master	Using removed V8 API	nodejs/node-gyp#1705

Feel free to edit this post to update the table.

[richardlau]

Duplicate of #25059. I like the table layout here but also prefer the title of #25059. We should probably pin whichever one we go with.

[targos]

@mcollina Sorry I missed your issue. I also prefer to have a table layout. I added a column with checkboxes.

[mcollina]

I’m ok of closing my issue but changing the title of this one.

[targos]

I added sqlite3, leveldown, level and iconv to the list. Someone should open an issue for the first 3.

[mcollina]

leveldown is fixed in the upcoming v5 release cc @ralphtheninja.