search

cytoscape / cytoscape.js-dagre

The Dagre layout for DAGs and trees for Cytoscape.js
MIT License
250 stars 83 forks source link

update dagre version #35

Closed YikaJ closed 6 years ago

YikaJ commented 6 years ago

34

maxkfranz commented 6 years ago

While the semver string in package.json lends itself to being upgraded to 0.8.2, it seems that the default version of npm (5.6.0, which comes with the latest v8 LTS) does not select the latest matching version on install. Instead, npm just installs the exact version string specified -- without regard for ^. Unfortunately, npm 6.4.1 doesn't resolve this either. A newer version of dagre can be installed by npm i dagre@0.8 without having to change the source of this extension.

Given that npm behaves in this way by default, I think it makes sense to merge this PR to make things simpler for package consumers.

Thanks for the PR.