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

Use the new sourceMappingURL syntax. #4

Merged
merged 1 commit into from
Jul 22, 2013

Conversation

russelldavis
Copy link
Contributor

@edc
Copy link
Owner

edc commented Jul 22, 2013

Thanks for sending this in.

@edc edc closed this Jul 22, 2013
@edc edc reopened this Jul 22, 2013
edc added a commit that referenced this pull request Jul 22, 2013
@edc edc merged commit f0596aa into edc:master Jul 22, 2013
@ashmoran
Copy link

I'm new to source maps so I've just been hacking away trying to get a build pipeline working. One thing I noticed is that the mapcat on npmjs.org is behind the GitHub source, so initially I installed in with npm from a cloned git repo.

After a while I discovered that Chrome wouldn't load my source maps regardless of the path in them, at which point I started Googling and discovered that this is Chrome's fault:

brunch/brunch#599

If it's true that "Chrome 29 will not land for at least 2 months", pushing this commit to npmjs.org may mysteriously break source maps for people using mapcat with Chrome. I don't know what the solution is, but I thought I should warn you about what I ran into.

@edc
Copy link
Owner

edc commented Aug 6, 2013

@ashmoran Thanks for the info. I will probably push the HEAD commit to npm after Chrome 29 then.

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.

3 participants