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

Added UMD definition to allow requireJs and CommonJs #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

togakangaroo
Copy link

Fix for this issue: #31

And fixed the gitignore file (it was not actually ignoring the bin directories)

@togakangaroo
Copy link
Author

Any movement on this?

@johnculviner
Copy link
Owner

I'm getting a "We can’t automatically merge this pull request." from Github and the diff has every single line changing so I've been hesitant to integrate it for sake of stability. I'll have to run it though a different diff tool than what Github has because I think the indents are confusing it. If we can get a diff that just shows the header and footer of the file changing (which is all it is right?) I can put it in right away. Sorry for the delay

@togakangaroo
Copy link
Author

Been a while but yeah, I'm pretty sure that's all it is. I'm using it in my current project.

Thanks for the response and thanks for the tool.

Sent from Mailbox for iPad

On Fri, Jan 24, 2014 at 10:40 AM, John Culviner notifications@github.com
wrote:

I'm getting a "We can’t automatically merge this pull request." from Github and the diff has every single line changing so I've been hesitant to integrate it for sake of stability. I'll have to run it though a different diff tool than what Github has because I think the indents are confusing it. If we can get a diff that just shows the header and footer of the file changing (which is all it is right?) I can put it in right away. Sorry for the delay

Reply to this email directly or view it on GitHub:
#32 (comment)

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

2 participants