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 source map paths #1496

Merged
merged 1 commit into from
Jun 21, 2015
Merged

Fix source map paths #1496

merged 1 commit into from
Jun 21, 2015

Conversation

martinvd
Copy link
Contributor

This sets the gulp-sourcemaps sourceRoot option, now pointing to the right locations of the source maps. Since it was not set, it was pointing to the default /source/ folder, which is not the right folder.

@austinpray austinpray added this to the 8.2.2 milestone Jun 18, 2015
@austinpray
Copy link
Contributor

I have some users on the forums testing this out. I'll test it out personally here in a second.

In the mean time: @swalkinshaw is probably going to make you squash your two commits into one. 👮

I've tagged this for release in 8.2.2 as well.

@martinvd
Copy link
Contributor Author

Thanks for your reply 👍 Sorry that I didn't do an interactive rebase btw!

@martinvd
Copy link
Contributor Author

I just squashed the two commits into one.

@austinpray
Copy link
Contributor

@retlehs good to go

retlehs added a commit that referenced this pull request Jun 21, 2015
@retlehs retlehs merged commit cd6095d into roots:master Jun 21, 2015
@austinpray
Copy link
Contributor

@martinvd
Copy link
Contributor Author

@austinpray That's interesting. I'll have a look into this asap and will try to reproduce it as well and then respond in the Roots Discourse issue.

@austinpray
Copy link
Contributor

Thanks bud!

@martinvd
Copy link
Contributor Author

np 👍

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