node-sass - npm Package Compare versions

Comparing version 1.1.3 to 1.1.4

build.js

@@ -0,0 +0,0 @@ #!/usr/bin/env node

CONTRIBUTING.md

@@ -0,0 +0,0 @@ ## Contributing

lib/render.js

@@ -1,6 +0,14 @@
-var sass = require('../sass'),
+var fs = require('fs'),
     chalk = require('chalk'),
-    fs = require('fs');
+    sass = require('../sass');
 
-function render(options, emitter) {
+/**
+ * Render
+ *
+ * @param {Object} options
+ * @param {Object} emitter
+ * @api public
+ */
+
+module.exports = function(options, emitter) {
   var renderOptions = {
@@ -32,3 +40,3 @@ imagePath: options.imagePath,
 
-    if (options.stdout || (!options.dest && (!process.stdout.isTTY || !process.env.isTTY))) {
+    if (options.stdout || (!options.dest && !process.stdout.isTTY)) {
       emitter.emit('log', css);
@@ -41,3 +49,6 @@ return done();
     fs.writeFile(options.dest, css, function(err) {
-      if (err) { return emitter.emit('error', chalk.red('Error: ' + err)); }
+      if (err) {
+        return emitter.emit('error', chalk.red('Error: ' + err));
+      }
+
       emitter.emit('warn', chalk.green('Wrote CSS to ' + options.dest));
@@ -50,4 +61,8 @@ emitter.emit('write', err, options.dest, css);
       todo++;
+
       fs.writeFile(options.sourceMap, sourceMap, function(err) {
-        if (err) {return emitter.emit('error', chalk.red('Error' + err)); }
+        if (err) {
+          return emitter.emit('error', chalk.red('Error' + err));
+        }
+
         emitter.emit('warn', chalk.green('Wrote Source Map to ' + options.sourceMap));
@@ -67,4 +82,2 @@ emitter.emit('write-source-map', err, options.sourceMap, sourceMap);
   sass.render(renderOptions);
-}
-
-module.exports = render;
+};

libsass/Readme.md

@@ -0,0 +0,0 @@ Libsass

package.json

@@ -5,3 +5,3 @@ {
   "description": "wrapper around libsass",
-  "version": "1.1.3",
+  "version": "1.1.4",
   "homepage": "https://github.com/sass/node-sass",
@@ -8,0 +8,0 @@ "keywords": [

README.md

@@ -0,0 +0,0 @@ # node-sass

sass.js

@@ -0,0 +0,0 @@ var path = require('path');

test/api.js

@@ -6,3 +6,4 @@ var assert = require('assert'),
     sass = require('../sass'),
-    fixture = path.join.bind(null, __dirname, 'fixtures');
+    fixture = path.join.bind(null, __dirname, 'fixtures'),
+    resolveFixture = path.resolve.bind(null, __dirname, 'fixtures');
 
@@ -13,7 +14,7 @@ describe('api (deprecated)', function() {
       var src = read(fixture('simple/index.scss'), 'utf8');
-      var expected = read(fixture('simple/expected.css'), 'utf8');
+      var expected = read(fixture('simple/expected.css'), 'utf8').trim();
 
       sass.render(src, function(err, css) {
         assert(!err);
-        assert.equal(css, expected);
+        assert.equal(css.trim(), expected);
         done();
@@ -34,5 +35,5 @@ });
       var src = read(fixture('simple/index.scss'), 'utf8');
-      var expected = read(fixture('simple/expected.css'), 'utf8');
+      var expected = read(fixture('simple/expected.css'), 'utf8').trim();
 
-      assert.equal(sass.renderSync(src), expected);
+      assert.equal(sass.renderSync(src).trim(), expected);
       done();
@@ -55,3 +56,3 @@ });
       var src = read(fixture('simple/index.scss'), 'utf8');
-      var expected = read(fixture('simple/expected.css'), 'utf8');
+      var expected = read(fixture('simple/expected.css'), 'utf8').trim();
 
@@ -61,3 +62,3 @@ sass.render({
         success: function(css) {
-          assert.equal(css, expected);
+          assert.equal(css.trim(), expected);
           done();
@@ -81,3 +82,3 @@ }
       var src = read(fixture('include-path/index.scss'), 'utf8');
-      var expected = read(fixture('include-path/expected.css'), 'utf8');
+      var expected = read(fixture('include-path/expected.css'), 'utf8').trim();
 
@@ -91,3 +92,3 @@ sass.render({
         success: function(css) {
-          assert.equal(css, expected);
+          assert.equal(css.trim(), expected);
           done();
@@ -98,22 +99,5 @@ }
 
-    it('should compile with include paths', function(done) {
-      var src = read(fixture('include-path/index.scss'), 'utf8');
-      var expected = read(fixture('include-path/expected.css'), 'utf8');
-
-      sass.render({
-        data: src,
-        includePaths: [
-          fixture('include-path/functions'),
-          fixture('include-path/lib')
-        ],
-        success: function(css) {
-          assert.equal(css, expected);
-          done();
-        }
-      });
-    });
-
     it('should compile with image path', function(done) {
       var src = read(fixture('image-path/index.scss'), 'utf8');
-      var expected = read(fixture('image-path/expected.css'), 'utf8');
+      var expected = read(fixture('image-path/expected.css'), 'utf8').trim();
 
@@ -124,3 +108,3 @@ sass.render({
         success: function(css) {
-          assert.equal(css, expected);
+          assert.equal(css.trim(), expected);
           done();
@@ -146,3 +130,3 @@ }
       var src = read(fixture('precision/index.scss'), 'utf8');
-      var expected = read(fixture('precision/expected.css'), 'utf8');
+      var expected = read(fixture('precision/expected.css'), 'utf8').trim();
 
@@ -153,3 +137,3 @@ sass.render({
         success: function(css) {
-          assert.equal(css, expected);
+          assert.equal(css.trim(), expected);
           done();
@@ -179,5 +163,5 @@ }
       var src = read(fixture('simple/index.scss'), 'utf8');
-      var expected = read(fixture('simple/expected.css'), 'utf8');
+      var expected = read(fixture('simple/expected.css'), 'utf8').trim();
 
-      assert.equal(sass.renderSync({ data: src }), expected);
+      assert.equal(sass.renderSync({ data: src }).trim(), expected);
       done();
@@ -199,3 +183,3 @@ });
       var dest = fixture('simple/build.css');
-      var expected = read(fixture('simple/expected.css'), 'utf8');
+      var expected = read(fixture('simple/expected.css'), 'utf8').trim();
 
@@ -206,3 +190,4 @@ sass.renderFile({
         success: function() {
-          assert.equal(read(dest, 'utf8'), expected);
+          assert.equal(read(dest, 'utf8').trim(), expected);
+          fs.unlinkSync(dest);
           done();
@@ -315,5 +300,5 @@ }
     it('should contain an array of all included files', function(done) {
-      assert.equal(stats.includedFiles[0], fixture('include-files/bar.scss'));
-      assert.equal(stats.includedFiles[1], fixture('include-files/foo.scss'));
-      assert.equal(stats.includedFiles[2], fixture('include-files/index.scss'));
+      assert.equal(stats.includedFiles[0], resolveFixture('include-files/bar.scss'));
+      assert.equal(stats.includedFiles[1], resolveFixture('include-files/foo.scss'));
+      assert.equal(stats.includedFiles[2], resolveFixture('include-files/index.scss'));
       done();
@@ -327,3 +312,3 @@ });
         success: function () {
-          assert.deepEqual(stats.includedFiles, [fixture('simple/index.scss')]);
+          assert.deepEqual(stats.includedFiles, [resolveFixture('simple/index.scss')]);
           done();
@@ -387,3 +372,3 @@ }
     it('should contain the given entry file', function(done) {
-      assert.equal(stats.entry, fixture('include-files/index.scss'));
+      assert.equal(stats.entry, resolveFixture('include-files/index.scss'));
       done();
@@ -393,5 +378,5 @@ });
     it('should contain an array of all included files', function(done) {
-      assert.equal(stats.includedFiles[0], fixture('include-files/bar.scss'));
-      assert.equal(stats.includedFiles[1], fixture('include-files/foo.scss'));
-      assert.equal(stats.includedFiles[2], fixture('include-files/index.scss'));
+      assert.equal(stats.includedFiles[0], resolveFixture('include-files/bar.scss'));
+      assert.equal(stats.includedFiles[1], resolveFixture('include-files/foo.scss'));
+      assert.equal(stats.includedFiles[2], resolveFixture('include-files/index.scss'));
       done();
@@ -398,0 +383,0 @@ });

test/cli.js

@@ -5,3 +5,3 @@ var assert = require('assert'),
     read = require('fs').readFileSync,
-    spawn = require('child_process').spawn,
+    spawn = require('cross-spawn'),
     cli = path.join(__dirname, '..', 'bin', 'node-sass'),
@@ -8,0 +8,0 @@ fixture = path.join.bind(null, __dirname, 'fixtures');

test/sass-spec/README.md

@@ -0,0 +0,0 @@ sass-spec

test/sass-spec/spec/basic/12_pseudo_classes_and_elements/notes.txt

@@ -0,0 +0,0 @@ Original Sass and libsass can produce different output for functional pseudo-classes. Original Sass does not normalize whitespace contained within an expression, and it also accepts invalid expressions. Libsass is stricter, and removes superfluous whitespace. E.g., on the following selector:

test/spec.js

@@ -0,0 +0,0 @@ var assert = require('assert'),

test/util/index.js

@@ -0,0 +0,0 @@ var fs = require('fs'),

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

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

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 1 instance in 1 package

Improved metrics

Number of low supply chain risk alerts

10

decreased by-16.67%

Number of medium supply chain risk alerts

2

decreased by-33.33%

Worsened metrics

Total package byte prevSize

21484602

decreased by-31.94%

Number of package files

1612

decreased by-0.12%

Lines of code

22046

decreased by-0.71%

No dependency changes