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

Generate source maps #59

Closed
wants to merge 1 commit into from

Conversation

duncanbeevers
Copy link
Contributor

  • Activated by the gulp sourceMap boolean.
  • Won't try to generate sourceMaps for handlebars versions < 3
  • Strips path information, only preserves filename.

Closes #50

@duncanbeevers
Copy link
Contributor Author

Regarding path stripping, this issue is open about absolute resourcePath in webpack. If something more relative-y comes along, it might makes sense to jump ship to that.

@@ -28,6 +29,7 @@
"devDependencies": {
"html-loader": "^0.3.0",
"mocha": "^1.21.4",
"semver": "^5.0.3",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't a devDep

@SimenB
Copy link
Contributor

SimenB commented Sep 23, 2015

Sweet! I haven't tried it, but how does debugging work? Currently I can put a breakpoint in the compiled template, does something like that work if using sourcemaps?

@pcardune
Copy link
Owner

pcardune commented Mar 9, 2016

@duncanbeevers, Thanks for the contribution. Can you add some docs to the README file explaining this new option?

@pcardune
Copy link
Owner

I'm going to close this since it has seen no movement. Reopen if you want it to get merge in!

@pcardune pcardune closed this May 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants