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

Cleanup context usage and extend code #2313

Merged
merged 3 commits into from
Feb 4, 2017
Merged

Conversation

mgreter
Copy link
Contributor

@mgreter mgreter commented Feb 4, 2017

Makes the API of extend a bit more sane

I removed ctx.options from some to_string occurences. The occurences were AFAIR only debug or comparison related (the comparison one should go once we adopt to sass/sass#2229). Also I can't think of any usage of numbers in selectors (maybe attributes?), so this seems legit to use default precision.

Also removes unused simpleSelectorOrderDependent from contains function.

@mgreter mgreter merged commit b31cf55 into master Feb 4, 2017
@nschonni nschonni deleted the cleanup/ctx-and-extend branch February 15, 2017 07:03
@mgreter mgreter self-assigned this Mar 30, 2017
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