Skip to content

Conversation

@nschonni
Copy link
Contributor

@nschonni nschonni commented Jun 28, 2019

Depends on sass/libsass#2933

@nschonni nschonni force-pushed the fix--Enable-libsass-2914 branch from f0c26ce to 9daa74f Compare June 28, 2019 03:48
@nex3 nex3 self-requested a review July 1, 2019 21:22
Copy link
Contributor

@nex3 nex3 left a comment

Choose a reason for hiding this comment

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

When you're making a spec PR that depends on LibSass changes, please add "[skip libsass]" to the PR description. This will tell Travis not to run tests against LibSass. Similarly, the LibSass PR description should include "See #1420" to tell LibSass to run specs against this PR, which verifies that your change does in fact fix these specs.

@nschonni
Copy link
Contributor Author

@nex3 the upstream PR landed now, if you want to clear the changes requested

@nex3
Copy link
Contributor

nex3 commented Jul 15, 2019

I'm landing this even though Travis is red because multiple changes (including this one) were landed in LibSass at once, so we need to land multiple fixes at once to get everything green again.

@nex3 nex3 merged commit a838a41 into sass:master Jul 15, 2019
@nschonni nschonni deleted the fix--Enable-libsass-2914 branch July 15, 2019 21:44
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.

2 participants