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

Which files need to be hosted? #30

Closed
kennynaoh opened this issue Aug 31, 2016 · 5 comments
Closed

Which files need to be hosted? #30

kennynaoh opened this issue Aug 31, 2016 · 5 comments

Comments

@kennynaoh
Copy link

kennynaoh commented Aug 31, 2016

Hello @giniedp ,
I am the member of cdnjs project.
We want to host this library.
But there is a question want to ask.
I think the main file are release/spritespin.js and release/spritespin.min.js on github.
But I am not sure does there any dependency files need to be grab under src/?
Or any other files need to be grab.
Please help me confirm that.
Thanks!

cdnjs/cdnjs#8721

@giniedp
Copy link
Owner

giniedp commented Aug 31, 2016

hi, thanks for the question. I was about to add spritespin to cdnjs but got stuck in my daily work and postponed my intend.
You are correct. The files in release directory are the only ones that needs to be hosted. They are generated from the files in src. The only external dependency is the jQuery framework.

@kennynaoh
Copy link
Author

Thanks!
But there are different file format before the version 3.2.1.
Do we also just grab these two files spritespin.js and spritespin.min.js under dist/?
Or we need to grab others under dist/?
Thanks!

@giniedp
Copy link
Owner

giniedp commented Sep 1, 2016

the dist folder was confusing for many people, so i decided to put the spritespin.js and the spritespin.min.js files into the release folder. Before that one had to grab these files from the dist folder, now its the release folder. But its always the spritespin.js and the spritespin.min.js. All other files are just usage examples.

@giniedp
Copy link
Owner

giniedp commented Sep 23, 2016

Thanks for putting it on cdnjs

@giniedp giniedp closed this as completed Sep 23, 2016
@kennynaoh
Copy link
Author

You are welcome

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

No branches or pull requests

2 participants