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

add sigma.js vs 1.0.2 #782

Merged
merged 1 commit into from
May 10, 2014
Merged

add sigma.js vs 1.0.2 #782

merged 1 commit into from
May 10, 2014

Conversation

pnommensen
Copy link
Contributor

This is a popular Javascript library for graphs. https://twitter.com/sigmajs

@jimaek please confirm it's correct to include /files/sigma.js/plugins as well? They are important and also ship with the release download https://github.com/jacomyal/sigma.js/releases .

The filenames with uppercase letters are from the original version, no alterations.

@jsdelivrbot
Copy link
Contributor

It looks like you want to contribute to jsdelivr/jsdelivr, @pnommensen, however there seems to be some issues with your pull request. See contributing for help ammending your PR...

  • Please see contributing before fixing your PR...
    • files/sigma.js/1.0.2/plugins/sigma.layout.forceAtlas2.min.js should be lowercase
    • files/sigma.js/1.0.2/plugins/sigma.plugins.dragNodes.min.js should be lowercase
    • files/sigma.js/1.0.2/plugins/sigma.renderers.customShapes.min.js should be lowercase
  • Might be nothing but I have some concerns about the files in your submission...
    • Unescaped unicode characters in sigma.min.js (at pos 118)

Thanks again for contributing.. If you think this review was wrong/unfair/etc. submit a bug on the bot's repo
Responsive is better than fast.

change order info.ini
@pnommensen pnommensen mentioned this pull request May 10, 2014
jimaek added a commit that referenced this pull request May 10, 2014
@jimaek jimaek merged commit dc2d3e9 into jsdelivr:master May 10, 2014
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.

None yet

3 participants