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

Add spec for issue 945 #276

Closed
wants to merge 1 commit into from
Closed

Add spec for issue 945 #276

wants to merge 1 commit into from

Conversation

am11
Copy link
Contributor

@am11 am11 commented Mar 15, 2015

This PR adds an error spec for issue described at sass/libsass#945.

@xzyfer
Copy link
Contributor

xzyfer commented Mar 15, 2015

Thanks @am11! Our spec format has changed slightly. Use https://github.com/sass/sass-spec/tree/master/spec/libsass-todo-issues/issue_823 as a reference.

@am11
Copy link
Contributor Author

am11 commented Mar 16, 2015

Ouch, I thought that only applies to the result producing specs. :)

@am11
Copy link
Contributor Author

am11 commented Mar 16, 2015

Rebased and re-pushed: 942a8bb.

@xzyfer
Copy link
Contributor

xzyfer commented Mar 16, 2015

Yeah on second thought I'm not sure how to handle this since we don't currently assert errors. I think it'll be one of those "Just trust us" cases for now :/

This was referenced Sep 20, 2015
@saper saper closed this Sep 21, 2015
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.

4 participants