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

fix: fix source maps #181

Merged
merged 1 commit into from
Mar 14, 2019
Merged

fix: fix source maps #181

merged 1 commit into from
Mar 14, 2019

Conversation

pskelin
Copy link
Contributor

@pskelin pskelin commented Mar 14, 2019

Template files with DOS line endings are breaking the source map.
Convert DOS line endings to Unix line endings when generating renderers.
This way, .hbs files with DOS line endings (no eslint) will not break the source
maps as the generated renderer will have the correct line ending.
The change is also correcting the line endings of existing .hbs templates.

Template files with DOS line endings are breaking the source map.
Convert DOS line endings to Unix line endings when generating renderers.
This way, .hbs files with DOS line endings (no eslint) will not break the source
maps as the generated renderer will have the correct line ending
Copy link
Member

@adrian-bobev adrian-bobev left a comment

Choose a reason for hiding this comment

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

Do we need .hbs files to be added in the .gitattributes or only replacement in the plugin is enough.

@pskelin
Copy link
Contributor Author

pskelin commented Mar 14, 2019

Do we need .hbs files to be added in the .gitattributes or only replacement in the plugin is enough.

we could add them for sure, but templates from other repositories that do not have the correct line ending should still be processed correctly by the conversion which is the more important aspect.

@pskelin pskelin merged commit 7084c96 into master Mar 14, 2019
@pskelin pskelin deleted the fix-source-maps branch March 14, 2019 12:39
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