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

suggestion to not use the sourceMapFileInline option #127

Closed
wants to merge 1 commit into from

Conversation

bassjobsen
Copy link

When searching for a solution for #126 i wondered why using Less with sourceMapFileInline (setting the SourceContent property of the source map with source files contents).

Not using the sourceMapFileInline and using result.map directly gives me the same results after some testing.

When searching for a solution for gulp-community#126 i wondered why using Less with sourceMapFileInline (setting the SourceContent property of the source map with source files contents).

Not using the sourceMapFileInline and using `result.map` directly gives me the same results after some testing.
@bassjobsen bassjobsen changed the title suggestion to no use the sourceMapFileInline option suggestion to not use the sourceMapFileInline option Jan 24, 2015
@bassjobsen
Copy link
Author

After all the the sourceMapFileInline option seems required indeed, https://www.npmjs.com/package/gulp-sourcemaps:

Including the content is the recommended way, because it "just works". When setting this to false you have to host the source files and set the correct sourceRoot.

@bassjobsen bassjobsen closed this Jan 24, 2015
@bassjobsen bassjobsen deleted the patch-1 branch January 24, 2015 15:03
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.

1 participant