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

Use jstransformer-scss and node-sass #4

Merged
merged 61 commits into from
Jun 6, 2016
Merged

Use jstransformer-scss and node-sass #4

merged 61 commits into from
Jun 6, 2016

Conversation

RobLoach
Copy link
Member

@RobLoach RobLoach commented Jul 19, 2015

Build upon jstransformer-scss, brings in Boilerplate for updated metadata.

TimothyGu and others added 30 commits April 2, 2015 21:12
This adds the inputFormats key to the boilerplate, allowing transformers to define what the expect input is:
https://github.com/jstransformers/jstransformer#inputformats
feat(inputFormats): Add inputFormats
This will make npm only deploy index.js and LICENSE.md.
Update and rename History.md to HISTORY.md
I believe it includes all `*.md` files automatically for us.
Remove "LICENSE.md" from "files" in package.json
```
   Output:	"<span class="foo" id='bar'>hello world</span>\n"
   Expected:	"<span class="foo" id='bar'>hello world</span>"
 ✓ escape-html.render() (11ms)
```
While the tests pass, there's an extra \n at the end of output console log. Having the input to assertEqual() there will fix it.
Disable email notifications
Fix tests to trim the output
@RobLoach
Copy link
Member Author

RobLoach commented Aug 5, 2015

@ForbesLindesay

Did this break the old test cases?

Probably... We should look through the tests and ensure the implementations are still there.

I think the scss module is native

To my knowledge, the sass package (which is what jstransformer-sass currently uses) is deprecated. It...

  • Lacks features from SASS and libsass
  • Was put together by tj years ago; don't think anyone is willing to keep it going
  • Has better implementations of a native JavaScript SASS-like in other systems like rework or postcss
  • Confuses people as when people use "sass", they expect SASS... Not an incomplete implementation of it.

@tunnckoCore tunnckoCore mentioned this pull request Aug 5, 2015
@tunnckoCore
Copy link
Member

I dont see the problem? Let's merge.

Im just confused about this SASS/SCSS-ish things. Actually the good packages is called node-sass, but it actually accept only scss syntax by default, and sass with indentedSyntax:true option.. that's .. nah.. "God bless good coders and their minds!"

@RobLoach
Copy link
Member Author

RobLoach commented Aug 5, 2015

I dont see the problem? Let's merge.

We should replicate the currently existing test into a new folder, so rather than replacing it, we keep the existing one alive and well.

Im just confused about this SASS/SCSS-ish things

SASS is the language. The file extension convention is usually .sass which indicates indentSyntax:true with the whitespace and .scss is Sassy-CSS, without the indentation.

:font-size 15px
+striped
=striped
tr
:background #fff
+large
Copy link
Member Author

Choose a reason for hiding this comment

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

The original test is now at test/table/input.sass. Mixins are declared by using =, they are used via +:
http://sass-lang.com/guide#topic-6

RobLoach and others added 12 commits August 12, 2015 23:14
Moved Boilerplate documentation to the Wiki
`npm` sometimes fixes the scripts to be alphabetized. This will avoid that change in the future.
Add Node.js 4.0.0 to Travis testing
Anyone using io.js should be on node 4.0.0 by now.
Add 'engines' spec into package.json
@RobLoach
Copy link
Member Author

The tests have been updated accordingly. Any thoughts?

@RobLoach RobLoach changed the title Update Inherit from jstransformer-scss and node-sass Jun 6, 2016
@RobLoach RobLoach changed the title Inherit from jstransformer-scss and node-sass Use jstransformer-scss and node-sass Jun 6, 2016
@RobLoach RobLoach merged commit ab7f8df into master Jun 6, 2016
@RobLoach RobLoach deleted the update branch June 6, 2016 15:58
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.

5 participants