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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SCSS support #169

Merged
merged 2 commits into from Nov 4, 2017
Merged

Add SCSS support #169

merged 2 commits into from Nov 4, 2017

Conversation

Olivia5k
Copy link
Contributor

@Olivia5k Olivia5k commented Nov 3, 2017

This might not cover the entire spec, but it covers my use cases!

Also, I couldn't run the tests because cask couldn't install. They are simple and from my understanding of the other definitions they should be at least mostly okay. 馃槄

This might not cover the entire spec, but it covers my use cases!
@coveralls
Copy link

coveralls commented Nov 3, 2017

Coverage Status

Coverage remained the same at 96.84% when pulling 0521270 on thiderman:master into e1140c3 on jacktasia:master.

@jacktasia
Copy link
Owner

Thanks for the PR! Looks like the tests are not passing though. Looks like you're setting the test value to what to jump from instead of an example/test match of the actual function to jump to.

I was able to fix them pretty easily: https://github.com/jacktasia/dumb-jump/compare/scss-tests -> https://travis-ci.org/jacktasia/dumb-jump/jobs/296997840 if you want to update your branch so you get credit for your work...or if you don't care I can just merge my branch. Up to you. Thanks again!

@coveralls
Copy link

coveralls commented Nov 4, 2017

Coverage Status

Coverage remained the same at 96.84% when pulling c0f1c7a on thiderman:master into e1140c3 on jacktasia:master.

@jacktasia
Copy link
Owner

Looks like I could push to your branch. Thanks!

@jacktasia jacktasia merged commit ab5c6cc into jacktasia:master Nov 4, 2017
netromdk pushed a commit to netromdk/dumb-jump that referenced this pull request Dec 2, 2017
* Add SCSS support

This might not cover the entire spec, but it covers my use cases!

* fix scss tests
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.

None yet

3 participants