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

Support sourceMap generation #63

Closed
wants to merge 4 commits into from

Conversation

scottydawg
Copy link
Contributor

This introduces support for sourceMap generation from the plugin. Please see the comments on the diff for implementation details.

@tkellen
Copy link
Member

tkellen commented Mar 20, 2013

We can address the join issue when jashkenas/coffeescript#2779 is resolved. I'll review this first thing tomorrow. Thanks again for the fantastic PRs!

@tkellen
Copy link
Member

tkellen commented Mar 21, 2013

This looks great @scottydawg, we just need to get some documentation in. Can you add the relevant details to the doc/* files? Our README is generated dynamically, so don't edit it directly.

@jrolfs
Copy link

jrolfs commented Mar 22, 2013

Just gonna go ahead and throw in a preemptive thank you for this. More than happy to lend a hand anywhere I can.

@tkellen tkellen closed this in 082d49a Mar 22, 2013
@tkellen
Copy link
Member

tkellen commented Mar 22, 2013

Incredible PR @scottydawg, and a beautiful refactoring of this task. Merged and published as 0.6.4--thank you!

@scottydawg
Copy link
Contributor Author

No worries - thanks for providing the plugin, it's been a huge help to our team. Glad I can contribute to it.

@dbrans dbrans mentioned this pull request Mar 28, 2013
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