less 3.6.0 → 3.8.1

package/CHANGELOG.md

@@ -1,3 +1,16 @@
+## 3.7.1
+
+2018-07-11
+  - Fixes mistake in `lessc` (using `console.warning` instead of actual `console.warn`)
+  - Adds `lessc` tests
+
+## 3.7.0
+
+2018-07-11
+  - New each() function! [See docs](http://lesscss.org/functions/#list-functions-each)
+  - New math modes! Also [see docs](http://lesscss.org/usage/#less-options-math).
+  - Bug fixes
+
 ## 3.6.0
 
 2018-07-10

package/Gruntfile.js

@@ -70,14 +70,18 @@ module.exports = function (grunt) {
 
     var sauceJobs = {};
 
-    var browserTests = [   'filemanager-plugin',
+
+    var browserTests = [
+        'filemanager-plugin',
         'visitor-plugin',
         'global-vars',
         'modify-vars',
         'production',
         'rootpath-relative',
+        'rootpath-rewrite-urls',
         'rootpath',
         'relative-urls',
+        'rewrite-urls',
         'browser',
         'no-js-errors',
         'legacy'
@@ -184,6 +188,21 @@ module.exports = function (grunt) {
             benchmark: {
                 command: 'node benchmark/index.js'
             },
+            opts: { // test running with all current options (using `opts` since `options` means something already)
+                command: [ // @TODO: make this more thorough
+                    // CURRENT OPTIONS
+                    // --math
+                    'node bin/lessc --math=always test/less/lazy-eval.less tmp/lazy-eval.css',
+                    'node bin/lessc --math=parens-division test/less/lazy-eval.less tmp/lazy-eval.css',
+                    'node bin/lessc --math=parens test/less/lazy-eval.less tmp/lazy-eval.css',
+                    'node bin/lessc --math=strict test/less/lazy-eval.less tmp/lazy-eval.css',
+                    'node bin/lessc --math=strict-legacy test/less/lazy-eval.less tmp/lazy-eval.css',
+
+                    // DEPRECATED OPTIONS
+                    // --strict-math
+                    'node bin/lessc --strict-math=on test/less/lazy-eval.less tmp/lazy-eval.css',
+                ].join(' && ')
+            },
             plugin: {
                 command: [
                     'node bin/lessc --clean-css="--s1 --advanced" test/less/lazy-eval.less tmp/lazy-eval.css',
@@ -194,12 +213,12 @@ module.exports = function (grunt) {
                     'node bin/lessc --plugin=clean-css="--s1 --advanced" --plugin=autoprefix="ie 11,Edge >= 13,Chrome >= 47,Firefox >= 45,iOS >= 9.2,Safari >= 9" test/less/lazy-eval.less tmp/lazy-eval.css'
                 ].join(' && ')
             },
-            'sourcemap-test': {
+            'sourcemap-test': { // quoted value doesn't seem to get picked up by time-grunt, or isn't output, at least; maybe just "sourcemap" is fine?
                 command: [
                     'node bin/lessc --source-map=test/sourcemaps/maps/import-map.map test/less/import.less test/sourcemaps/import.css',
                     'node bin/lessc --source-map test/less/sourcemaps/basic.less test/sourcemaps/basic.css'
                 ].join(' && ')
-            }
+            },
         },
 
         browserify: {
@@ -273,6 +292,7 @@ module.exports = function (grunt) {
                 keepRunner: true,
                 host: 'http://localhost:8081/',
                 vendor: [
+                    './node_modules/phantomjs-polyfill-object-assign/object-assign-polyfill.js',
                     'test/browser/vendor/promise.js',
                     'test/browser/jasmine-jsreporter.js',
                     'test/browser/common.js',
@@ -347,6 +367,14 @@ module.exports = function (grunt) {
                     outfile: 'tmp/browser/test-runner-relative-urls.html'
                 }
             },
+            rewriteUrls: {
+                src: ['test/browser/less/rewrite-urls/*.less'],
+                options: {
+                    helpers: 'test/browser/runner-rewrite-urls-options.js',
+                    specs: 'test/browser/runner-rewrite-urls-spec.js',
+                    outfile: 'tmp/browser/test-runner-rewrite-urls.html'
+                }
+            },
             rootpath: {
                 src: ['test/browser/less/rootpath/*.less'],
                 options: {
@@ -363,6 +391,14 @@ module.exports = function (grunt) {
                     outfile: 'tmp/browser/test-runner-rootpath-relative.html'
                 }
             },
+            rootpathRewriteUrls: {
+                src: ['test/browser/less/rootpath-rewrite-urls/*.less'],
+                options: {
+                    helpers: 'test/browser/runner-rootpath-rewrite-urls-options.js',
+                    specs: 'test/browser/runner-rootpath-rewrite-urls-spec.js',
+                    outfile: 'tmp/browser/test-runner-rootpath-rewrite-urls.html'
+                }
+            },
             production: {
                 src: ['test/browser/less/production/*.less'],
                 options: {
@@ -437,9 +473,6 @@ module.exports = function (grunt) {
 
     require('jit-grunt')(grunt);
 
-    // Actually load this plugin's task(s).
-    grunt.loadTasks('build/tasks');
-
     // by default, run tests
     grunt.registerTask('default', [
         'test'
@@ -503,6 +536,7 @@ module.exports = function (grunt) {
         'clean',
         'eslint',
         'shell:test',
+        'shell:opts',
         'shell:plugin',
         'browsertest'
     ];
@@ -518,6 +552,9 @@ module.exports = function (grunt) {
     // Run all tests
     grunt.registerTask('test', testTasks);
 
+    // Run shell option tests (includes deprecated options)
+    grunt.registerTask('shell-options', ['shell:opts']);
+
     // Run shell plugin test
     grunt.registerTask('shell-plugin', ['shell:plugin']);
 

package/bin/lessc

@@ -4,6 +4,7 @@ var path = require('path'),
     fs = require('../lib/less-node/fs'),
     os = require('os'),
     utils = require('../lib/less/utils'),
+    Constants = require('../lib/less/constants'),
     errno,
     mkdirp;
 
@@ -475,14 +476,42 @@ function processPluginQueue() {
                     options.rootpath = match[2].replace(/\\/g, '/');
                 }
                 break;
-            case 'ru':
             case 'relative-urls':
-                options.relativeUrls = true;
+                console.warn('The --relative-urls option has been deprecated. Use --rewrite-urls=all.');
+                options.rewriteUrls = Constants.RewriteUrls.ALL;
+                break;
+            case 'ru':
+            case 'rewrite-urls':
+                var m = match[2];
+                if (m) {
+                    if (m === 'local') {
+                        options.rewriteUrls = Constants.RewriteUrls.LOCAL;
+                    } else if (m === 'off') {
+                        options.rewriteUrls = Constants.RewriteUrls.OFF;
+                    } else if (m === 'all') {
+                        options.rewriteUrls = Constants.RewriteUrls.ALL;
+                    } else {
+                        console.error('Unknown rewrite-urls argument ' + m);
+                        continueProcessing = false;
+                        process.exitCode = 1;
+                    }
+                } else {
+                    options.rewriteUrls = Constants.RewriteUrls.ALL;
+                }
                 break;
             case 'sm':
             case 'strict-math':
+                console.warn('The --strict-math option has been deprecated. Use --math=strict.');
+                if (checkArgFunc(arg, match[2])) {
+                    if (checkBooleanArg(match[2])) {
+                        options.math = Constants.Math.STRICT_LEGACY;
+                    }
+                }
+                break;
+            case 'm':
+            case 'math':
                 if (checkArgFunc(arg, match[2])) {
-                    options.strictMath = checkBooleanArg(match[2]);
+                    options.math = match[2];
                 }
                 break;
             case 'su':