search

glejeune / node-graphviz

Node.js interface to the GraphViz graphing tool
http://algorithmique.net
MIT License
197 stars 58 forks source link

Fix missing error handling #26

Closed BridgeAR closed 6 years ago

BridgeAR commented 6 years ago

The fs calls here swallow errors and that behavior is long deprecated. This will throw from Node.js 10.x on.

It will from now on report the errors back to the error callback.

Reference: https://github.com/nodejs/node/pull/18668#

BridgeAR commented 6 years ago

Ping @glejeune