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

SAT.js #6642 #6643

Closed
wants to merge 5 commits into from
Closed

SAT.js #6642 #6643

wants to merge 5 commits into from

Conversation

goldenratio
Copy link

SAT.js #6642

@PeterDaveHello
Copy link
Contributor

Please also add auto-update config, and squash/fixup these commits into only one commit, thanks.

@goldenratio
Copy link
Author

I need help with squash.

@goldenratio
Copy link
Author

added auto updater

@PeterDaveHello
Copy link
Contributor

I need help with squash.

What kind of help do you need?

@maruilian11
Copy link
Contributor

@goldenratio you can use git rebase -i HEAD~5 👍

@goldenratio
Copy link
Author

do I have to clone the whole project for that?

@PeterDaveHello
Copy link
Contributor

Yes, please take a look at: sparseCheckout.md, thanks.

@goldenratio
Copy link
Author

Sorry don't have time for it. You can decline the PR.

I think there should be better solution to add JS lib to CDN. Probably a web interface with required input fields, and all the PR magic is done under the hood.

On 28 Jan 2016 14:20, "Peter Dave Hello" notifications@github.com wrote:

Yes, please take a look at: sparseCheckout.md
https://github.com/cdnjs/cdnjs/blob/master/documents/sparseCheckout.md,
thanks.


Reply to this email directly or view it on GitHub
#6643 (comment).

@PeterDaveHello
Copy link
Contributor

@goldenratio No problem, we can add by our side if this lib fits our requirement, please wait, thanks.

@maruilian11 maruilian11 mentioned this pull request Feb 1, 2016
8 tasks
@PeterDaveHello
Copy link
Contributor

Hello, any updates here? Thanks.

@PeterDaveHello
Copy link
Contributor

Any updates here? Thanks!

1 similar comment
@PeterBot
Copy link
Contributor

Any updates here? Thanks!

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.

4 participants