search

fedwiki / wiki-server

Federated Wiki client and server in Node.js
Other
153 stars 35 forks source link

Cleaning up a few dependencies #146

Closed paul90 closed 5 years ago

paul90 commented 5 years ago

This is partly a follow on from the event-stream incident.

As well as removing the dependency on event-stream, by re-writing the /system/factories.json get, this takes a look at the other dependencies - removing nsp and other that at not needed, and updating the test.

A few of the tests needed a slight change so they don't mix promises and callbacks, N.B. this requires Node.js 8, or greater. A new test is added for /system/factories.json.

Here we also update to using the latest version of coffeescript.

Still a few things to look at before this will be ready to merge...

WardCunningham commented 5 years ago

I thank you for your initiative in this regard.

WardCunningham commented 5 years ago

I'm happy to see this merged.