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

Adds support for Spaces #138

Merged
merged 1 commit into from
Jul 29, 2016
Merged

Conversation

IanMitchell
Copy link
Contributor

@IanMitchell IanMitchell commented Jul 27, 2016

Per this tweet it seems like there wasn't a conscious decision to not allow op variable syntax (for example >= phone) so I went ahead and took a pass at implementing it. Full credit to Hugo for the trim and first-index methods - I ripped them straight out of sass-semver. First time I've really done something intense in Sass, but gave it my best shot - please let me know if I broke styleguide somewhere 😄

@KittyGiraudel
Copy link
Collaborator

To be fair, you did even better than I would have. I’ll let @eduardoboucas have the last call and merge but for me it’s beyond perfect.

@IanMitchell
Copy link
Contributor Author

It's literally your code that I copy-pasted, so you gotta give yourself more credit than that haha.

@eduardoboucas
Copy link
Owner

Ditto! Really happy with it. Thanks @IanMitchell!

Probably won't have a chance to merge and release today, but will do that first thing tomorrow.

@IanMitchell
Copy link
Contributor Author

No rush! Thanks for the library :)

@eduardoboucas eduardoboucas merged commit 18d90d1 into eduardoboucas:master Jul 29, 2016
@eduardoboucas
Copy link
Owner

Done! 🎉

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