Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nodejs compatibility #4 #5

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

mef
Copy link

@mef mef commented Jun 12, 2016

made compatible with node.js #4 😄 using UMD (Universal Module Definition) returnExports pattern

  • client-side usage unchanged
  • documented inside README file
  • added example for usage node.js

A further enhancement could be to publish the module in npm, may be done separately from this pull request.

mef and others added 7 commits June 12, 2016 15:23
while keeping client-side usage unchanged
using UMD (Universal Module Definition) returnExports pattern https://github.com/umdjs/umd
apparently, github doesn't parse titles without space after the pound key anymore.
avoid hard-coded http scheme so that d3 doesn't get blocked as mixed active content
@philippefutureboy
Copy link

@upphiminn Any update on this?

@philippefutureboy
Copy link

Nevermind that - https://github.com/jgranstrom/jLouvain

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants