node-sass - npm Package Compare versions

Comparing version 0.9.6 to 1.0.0

lib/cli.js

@@ -1,2 +0,2 @@
-var watch = require('node-watch'),
+var watch = require('node-watch'),
     render = require('./render'),
@@ -52,2 +52,7 @@ path = require('path'),
   })
+  .options('indented-syntax', {
+      describe: 'Treat data from stdin as sass code (versus scss)',
+      type: 'boolean',
+      alias: 'i'
+  })
   .options('help', {
@@ -97,9 +102,3 @@ describe: 'Print usage info',
 
-  if (options.sourceComments === 'map' && !options.sourceMap) {
-    options.sourceMap = true;
-  }
-
   if (options.sourceMap) {
-    options.sourceComments = 'map';
-
     if (options.sourceMap === true) {
@@ -145,2 +144,3 @@ options.sourceMap = options.dest + '.map';
     omitSourceMapUrl: argv['omit-source-map-url'],
+    indentedSyntax: argv['indented-syntax'],
     outputStyle: argv['output-style'],
@@ -147,0 +147,0 @@ precision: argv.precision,

lib/render.js

@@ -10,2 +10,3 @@ var sass = require('../sass'),
     omitSourceMapUrl: options.omitSourceMapUrl,
+    indentedSyntax: options.indentedSyntax,
     outFile: options.outFile,
@@ -12,0 +13,0 @@ outputStyle: options.outputStyle,

libsass/Readme.md

@@ -6,3 +6,5 @@ Libsass
 
-[![Build Status](https://travis-ci.org/sass/libsass.png?branch=master)](https://travis-ci.org/sass/libsass) [![Bountysource](https://www.bountysource.com/badge/tracker?tracker_id=283068)](https://www.bountysource.com/trackers/283068-libsass?utm_source=283068&utm_medium=shield&utm_campaign=TRACKER_BADGE)
+[![Build Status](https://travis-ci.org/sass/libsass.png?branch=master)](https://travis-ci.org/sass/libsass)
+[![Bountysource](https://www.bountysource.com/badge/tracker?tracker_id=283068)](https://www.bountysource.com/trackers/283068-libsass?utm_source=283068&utm_medium=shield&utm_campaign=TRACKER_BADGE)
+[![Coverage Status](https://img.shields.io/coveralls/sass/libsass.svg)](https://coveralls.io/r/sass/libsass?branch=feature%2Ftest-travis-ci-3)
 
@@ -16,2 +18,4 @@ https://github.com/sass/libsass
 
+LibSass requires GCC 4.7+ or Clang/LLVM. If your OS is older, this version may not compile.
+
 About
@@ -45,4 +49,4 @@ -----
 
-Usage
------
+Library Usage
+-------------
 
@@ -88,2 +92,8 @@ While libsass is primarily implemented in C++, it provides a simple
 
+Implementations
+---------------
+
+For a full list of implementations, checkout our
+[implementations wiki page](https://github.com/sass/libsass/wiki/Implementations)
+
 About Sass
@@ -118,2 +128,5 @@ ----------
 
+sass2scss was originally written by Marcel Greter [@mgreter]
+and he happily agreed to have it merged into the project.
+
 [sass_interface.h]: sass_interface.h

package.json

@@ -5,3 +5,3 @@ {
   "description": "wrapper around libsass",
-  "version": "0.9.6",
+  "version": "1.0.0",
   "homepage": "https://github.com/sass/node-sass",
@@ -11,3 +11,6 @@ "keywords": [
     "css",
-    "libsass"
+    "libsass",
+    "scss",
+    "style",
+    "preprocessor"
   ],
@@ -51,3 +54,2 @@ "bugs": {
     "sinon": "~1.10.3",
-    "node-sass-middleware": "~0.3.0",
     "yargs": "~1.3.1",
@@ -54,0 +56,0 @@ "get-stdin": "~3.0.0"

README.md

@@ -6,2 +6,3 @@ # node-sass
 [![Build Status](https://travis-ci.org/sass/node-sass.svg?branch=master&style=flat)](https://travis-ci.org/sass/node-sass)
+[![Build status](https://ci.appveyor.com/api/projects/status/22mjbk59kvd55m9y/branch/master)](https://ci.appveyor.com/project/sass/node-sass/branch/master)
 [![npm version](https://badge.fury.io/js/node-sass.svg)](http://badge.fury.io/js/node-sass)
@@ -168,19 +169,6 @@ [![Dependency Status](https://david-dm.org/sass/node-sass.svg?theme=shields.io)](https://david-dm.org/sass/node-sass)
 
-Recompile `.scss` files automatically for connect and express based http servers
+Recompile `.scss` files automatically for connect and express based http servers.
 
-```javascript
-var server = connect.createServer(
-  sass.middleware({
-      src: __dirname
-    , dest: __dirname + '/public'
-    , debug: true
-    , outputStyle: 'compressed'
-    , prefix:  '/prefix'
-  }),
-  connect.static('/prefix', __dirname + '/public')
-);
-```
+This functionality has been moved to [`node-sass-middleware`](https://github.com/sass/node-sass-middleware) in node-sass v1.0.0
 
-Heavily inspired by <https://github.com/LearnBoost/stylus>
-
 ### DocPad Plugin
@@ -187,0 +175,0 @@

sass.js

@@ -35,6 +35,6 @@ var path = require('path');
 var SASS_SOURCE_COMMENTS = {
-  none: 0,
-  normal: 1,
-  'default': 1,
-  map: 2
+  none: false,
+  normal: true,
+  default: false,
+  map: true
 };
@@ -85,4 +85,5 @@
     style: SASS_OUTPUT_STYLE[options.output_style || options.outputStyle] || 0,
-    comments: SASS_SOURCE_COMMENTS[sourceComments] || 0,
+    comments: SASS_SOURCE_COMMENTS[sourceComments] || false,
     omitSourceMapUrl: options.omitSourceMapUrl,
+    indentedSyntax: options.indentedSyntax,
     stats: stats,
@@ -203,2 +204,4 @@ sourceMap: sourceMap,
 
-exports.middleware = require('node-sass-middleware');
+exports.middleware = function() {
+  return new Error('The middleware has been moved to https://github.com/sass/node-sass-middleware');
+};

test/cli.js

@@ -11,6 +11,6 @@ var path   = require('path'),
 
-var expectedSampleCompressed = '#navbar{width:80%;height:23px;}\
-#navbar ul{list-style-type:none;}\
-#navbar li{float:left;}\
-#navbar li a{font-weight:bold;}';
+var expectedSampleCompressed = '#navbar{width:80%;height:23px}\
+#navbar ul{list-style-type:none}\
+#navbar li{float:left}\
+#navbar li a{font-weight:bold}';
 
@@ -68,2 +68,13 @@ var expectedSampleNoComments = '#navbar {\n\
 
+  it('should treat data as indented code (.sass) if --indented-syntax flag is used', function(done) {
+    this.timeout(6000);
+    var src = fs.createReadStream(path.join(__dirname, 'indented.sass'));
+    var emitter = spawn(cliPath, ['--stdout', '--indented-syntax']);
+
+    // when hit the callback in the following,
+    // it means that data is recieved, so we are ok to go.
+    emitter.stdout.on('data', function() { done(); }); 
+    src.pipe(emitter.stdin);
+  });
+
   it('should print help when run with no arguments', function(done) {
@@ -263,2 +274,14 @@ var env = assign(process.env, { isTTY: true });
   });
+
+  it('should omit a sourceMappingURL from CSS if --omit-source-map-url flag is used', function(done) {
+    var emitter = cli([sampleScssPath, '--source-map', path.join(__dirname, '../sample.map'), '--omit-source-map-url']);
+    emitter.on('error', done);
+    emitter.on('done', function() {
+      fs.exists(sampleCssOutputPath, function(exists) {
+        assert.ok(fs.readFileSync(sampleCssOutputPath, 'utf8').indexOf('sourceMappingURL=') === -1);
+        if (exists) {fs.unlinkSync(sampleCssOutputPath);}
+        done();
+      });
+    });
+  });
 });

test/sass-spec.js

@@ -33,3 +33,3 @@ var assert = require('assert'),
     var suites = fs.readdirSync(suitesPath);
-    var ignoreSuites = ['todo', 'benchmarks'];
+    var ignoreSuites = ['libsass-todo-issues', 'libsass-todo-tests'];
 
@@ -47,18 +47,23 @@ suites.forEach(function(suite) {
           var testPath = path.join(suitePath, test);
+          var inputFilePath = path.join(testPath, 'input.scss');
 
-          it(test, function(done) {
-            sass.render({
-              file: path.join(testPath, 'input.scss'),
-              includePaths: [testPath, path.join(testPath, 'sub')],
-              success: function(css) {
-                var expected = fs.readFileSync(path.join(testPath, 'expected_output.css'), 'utf-8');
+          if (fs.existsSync(inputFilePath)) {
+            it(test, function(done) {
+              sass.render({
+                file: inputFilePath,
+                includePaths: [testPath, path.join(testPath, 'sub')],
+                success: function(css) {
+                  var expected = fs.readFileSync(path.join(testPath, 'expected_output.css'), 'utf-8');
 
-                assert.equal(normalize(css), normalize(expected));
-                done();
-              },
-              error: function(error) {
-                done(new Error(error));
-              }
+                  assert.equal(normalize(css), normalize(expected));
+                  done();
+                },
+                error: function(error) {
+                  done(new Error(error));
+                }
+              });
             });
-          });
+          } else {
+            it(test);
+          }
         });
@@ -65,0 +70,0 @@ });

test/sass-spec/README.md

 sass-spec
 =========
 
-A test suite for Sass. The test cases are all in the `/spec` folder
-and are organized, primarily, for the use of libsass development.
+A test suite for Sass. The test cases are all in the `/spec` folder.
 
-All tests in the `spec/todo` folder are expected to fail with libsass.
+Run tests against Ruby Sass with the `sass-spec.rb` file in the root directory.
 
-Run tests with the `sass-spec.rb` file in the root directory.
-
     ./sass-spec.rb 
 
 Full text help is available if you run that w/ the help options.
+
+## Organization
+
+The tests are organized this way:
+
+	* basic - The core tests taken from Sass' early development
+	* scss - The tests suite written for the introduction of scss
+	* libsass-open-issues - Tests for known libsass breakages. These are not run automatically.
+	* libsass-closed-issues - Tests for closed issues in the libsass directory.
+ 	* maps - Testing maps
+	* extends - Testing extends
+	* libsass-todo - Tests taken from Ruby Sass and moved over here, that do not pass in libsass yet.
+
+## Ruby Sass
+
+All tests with scss files named `input.disabled.scss` should be for non-supported tests.
+
+Ruby 2.1.0 contained a regression that changed the order of some selectors, causing test failures in sass-spec. That was fixed in Ruby 2.1.1. If you're running sass-spec against a Ruby Sass, please be sure not to use Ruby 2.1.0.
+
+## LibSass
+
+After installing a libsass dev enviroment (see libsass readme... sassc, this spec, and libsass), the tests are run by going
+to the libsass folder and running ./script/spec.
+
+## Contribution
+
+This project needs maintainers! There will be an ongoing process of simplifying test cases, reporting new issues and testing them here, and managing mergers of official test cases.
+
+This project requires help with the Ruby test drivers (better output, detection modes, etc) AND just with managing the issues and writing test cases.

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Copyleft License

License

(Experimental) Copyleft license information was found.

Found 1 instance in 1 package

Mixed license

License

(Experimental) Package contains multiple licenses.

Found 1 instance in 1 package

Non-permissive License

License

(Experimental) A license not known to be considered permissive was found.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Copyleft License

License

(Experimental) Copyleft license information was found.

Found 1 instance in 1 package

Mixed license

License

(Experimental) Package contains multiple licenses.

Found 1 instance in 1 package

No v1

Quality

Package is not semver >=1. This means it is not stable and does not support ^ ranges.

Found 1 instance in 1 package

Non-permissive License

License

(Experimental) A license not known to be considered permissive was found.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

21316018

increased by5.3%

Dependency count

10

decreased by-9.09%

Number of low quality alerts

0

decreased by-100%

Worsened metrics

Number of package files

1584

decreased by-9.85%

Lines of code

22350

decreased by-0.54%

Number of lines in readme file

275

decreased by-4.18%

Number of medium supply chain risk alerts

4

increased by33.33%

Dependency changes

• - Removednode-sass-middleware@~0.3.0

• - Removednode-sass-middleware@0.3.1(transitive)