less 3.0.0-pre.2 → 3.0.1

package/.eslintrc.json

@@ -0,0 +1,88 @@
+{
+  "env": {
+    "node": true
+  },
+  "globals": {},
+  "rules": {
+    "no-eval": 2,
+    "no-use-before-define": [
+      2,
+      {
+        "functions": false
+      }
+    ],
+    "no-undef": 0,
+    "no-unused-vars": 1,
+    "no-caller": 2,
+    "no-eq-null": 1,
+    "guard-for-in": 2,
+    "no-implicit-coercion": [
+      2,
+      {
+        "boolean": false,
+        "string": true,
+        "number": true
+      }
+    ],
+    "no-with": 2,
+    "no-mixed-spaces-and-tabs": 2,
+    "no-multiple-empty-lines": 2,
+    "dot-location": [
+      2,
+      "property"
+    ],
+    "operator-linebreak": [
+      0,
+      "after"
+    ],
+    "keyword-spacing": [
+      2,
+      {}
+    ],
+    "space-unary-ops": [
+      2,
+      {
+        "words": false,
+        "nonwords": false
+      }
+    ],
+    "no-spaced-func": 2,
+    "space-before-function-paren": [
+      1,
+      {
+        "anonymous": "ignore",
+        "named": "never"
+      }
+    ],
+    "comma-dangle": [
+      2,
+      "never"
+    ],
+    "no-trailing-spaces": 0,
+    "max-len": [
+      2,
+      160
+    ],
+    "comma-style": [
+      2,
+      "last"
+    ],
+    "curly": [
+      2,
+      "all"
+    ],
+    "space-infix-ops": 2,
+    "spaced-comment": 1,
+    "space-before-blocks": [
+      2,
+      "always"
+    ],
+    "indent": [
+      2,
+      4,
+      {
+        "SwitchCase": 1
+      }
+    ]
+  }
+}

package/.github/stale.yml

@@ -0,0 +1,17 @@
+# Number of days of inactivity before an issue becomes stale
+daysUntilStale: 120
+# Number of days of inactivity before a stale issue is closed
+daysUntilClose: 14
+# Issues with these labels will never be considered stale
+exemptLabels:
+  - up-for-grabs
+  - bug
+# Label to use when marking an issue as stale
+staleLabel: stale
+# Comment to post when marking an issue as stale. Set to `false` to disable
+markComment: >
+  This issue has been automatically marked as stale because it has not had
+  recent activity. It will be closed if no further activity occurs. Thank you
+  for your contributions.
+# Comment to post when closing a stale issue. Set to `false` to disable
+closeComment: false

package/.project

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<projectDescription>
+	<name>less.js</name>
+	<comment>Project less.js created by Buildship.</comment>
+	<projects>
+	</projects>
+	<buildSpec>
+		<buildCommand>
+			<name>org.eclipse.buildship.core.gradleprojectbuilder</name>
+			<arguments>
+			</arguments>
+		</buildCommand>
+	</buildSpec>
+	<natures>
+		<nature>org.eclipse.buildship.core.gradleprojectnature</nature>
+	</natures>
+</projectDescription>

package/.settings/org.eclipse.buildship.core.prefs

@@ -0,0 +1,2 @@
+#Wed Feb 14 13:51:01 PST 2018
+connection.project.dir=

package/.travis.yml

@@ -1,12 +1,31 @@
 language: node_js
+cache:
+  directories:
+    - travis-phantomjs
 node_js:
+  - "9"
+  - "8"
   - "6"
   - "4"
-  - "0.12"
-  - "0.10"
+before_install:
+  # from https://github.com/travis-ci/travis-ci/issues/3225#issuecomment-177592725
+  # and also from https://github.com/travis-ci/travis-ci/issues/3225#issuecomment-200965782
+  - phantomjs --version
+  - export PATH=$PWD/travis-phantomjs/phantomjs-2.1.1-linux-x86_64/bin:$PATH
+  - phantomjs --version
+  # Clear cache and download new copy of PhantomJS if the current version doesn't match 2.1.1.
+  - "if [ $(phantomjs --version) != '2.1.1' ]; then rm -rf $PWD/travis-phantomjs; mkdir -p $PWD/travis-phantomjs; fi"
+  - "if [ $(phantomjs --version) != '2.1.1' ]; then wget https://bitbucket.org/ariya/phantomjs/downloads/phantomjs-2.1.1-linux-x86_64.tar.bz2 -O $PWD/travis-phantomjs/phantomjs-2.1.1-linux-x86_64.tar.bz2; fi"
+  - "if [ $(phantomjs --version) != '2.1.1' ]; then tar -xvf $PWD/travis-phantomjs/phantomjs-2.1.1-linux-x86_64.tar.bz2 -C $PWD/travis-phantomjs; fi"
+  - phantomjs --version
 install:
   - npm install -g grunt-cli
-  - npm install
+  # node 0.10 & 0.12 have race condition issues when running custom install scripts
+  # this can cause phantomjs-prebuilt install script to fail with the error:
+  # <Cannot find module 'boom'>
+  # Seems related to: https://github.com/npm/npm/issues/8152
+  # using <travis_retry> solves this.
+  - travis_retry npm install
 env:
   global:
     - PHANTOMJS_CDNURL=http://cnpmjs.org/downloads

package/.vscode/launch.json

@@ -0,0 +1,14 @@
+{
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "type": "node",
+            "request": "launch",
+            "name": "Launch Program",
+            "program": "${workspaceFolder}/index"
+        }
+    ]
+}

package/CHANGELOG.md

@@ -1,3 +1,30 @@
+# 3.0.0
+2018-02-10
+  - Fix `calc()` function to not do math operations on compile
+  - Rename Directive -> AtRule & Rule -> Declaration 
+  - Cross-platform `@plugin` loading! (Node & Browser)
+  - Numerous changes / improvements to plugin architecture
+  - Simplified API calls in plugins (`less.atrule()` vs `new less.tree.AtRule()`)
+  - Property accessors (`$width` to refer to `width: 300px` value)
+  - Inline JavaScript disabled by default for security reasons (use `@plugin`)
+  - Improvements in Less error reporting
+  - Added feature: returning `null` / `false` from Less functions will remove that line
+  - Simple `boolean()` and `if()` functions added
+  - Bug fixes
+  - Removal of unnecessary nodes from API (like IE's `alpha()`)
+
+# 2.7.3
+2017-10-23
+
+  - Bump `request` dependency
+
+# 2.7.2
+2017-01-04
+
+  - Revert breaking changes to contrast() function
+  - Fix error reporting of lessc executable
+  - Changed octals to hex for ES6 strict mode
+
 # 2.7.1 HOTFIX
 
 2016-05-09

package/Gruntfile.js

@@ -2,10 +2,154 @@
 
 module.exports = function (grunt) {
 
+
+    grunt.option('stack', true)
+
     // Report the elapsed execution time of tasks.
     require('time-grunt')(grunt);
 
-    var COMPRESS_FOR_TESTS = true;
+    var COMPRESS_FOR_TESTS = false;
+    var git = require('git-rev');
+
+   // Sauce Labs browser
+    var browsers = [
+        // Desktop browsers
+        {
+            browserName: "chrome",
+            version: 'latest',
+            platform: 'Windows 7'
+        },
+        {
+            browserName: "firefox",
+            version: 'latest',
+            platform: 'Linux'
+        },
+        {
+            browserName: 'safari',
+            version: '9',
+            platform: 'OS X 10.11'
+        },
+        {
+            browserName: "internet explorer",
+            version: '8',
+            platform: 'Windows XP'
+        },
+        {
+            browserName: "internet explorer",
+            version: '11',
+            platform: 'Windows 8.1'
+        },
+        {
+            browserName: "edge",
+            version: '13',
+            platform: 'Windows 10'
+        },
+        // Mobile browsers
+        {
+            browserName: 'ipad',
+            deviceName: 'iPad Air Simulator',
+            deviceOrientation: 'portrait',
+            version: '8.4',
+            platform: 'OS X 10.9'
+        },
+        {
+            browserName: 'iphone',
+            deviceName: 'iPhone 5 Simulator',
+            deviceOrientation: 'portrait',
+            version: '9.3',
+            platform: 'OS X 10.11'
+        },
+        {
+            browserName: 'android',
+            deviceName: 'Google Nexus 7 HD Emulator',
+            deviceOrientation: 'portrait',
+            version: '4.4',
+            platform: 'Linux'
+        }
+    ];
+
+    var sauceJobs = {};
+
+    var browserTests = [   "filemanager-plugin",
+        "visitor-plugin",
+        "global-vars",
+        "modify-vars",
+        "production",
+        "rootpath-relative",
+        "rootpath",
+        "relative-urls",
+        "browser",
+        "no-js-errors",
+        "legacy"
+    ];
+
+    function makeJob(testName) {
+        sauceJobs[testName] = {
+            options: {
+                urls: testName === 'all' ?
+                    browserTests.map(function(name) {
+                        return "http://localhost:8081/tmp/browser/test-runner-" + name + ".html";
+                    }) :
+                    ["http://localhost:8081/tmp/browser/test-runner-" + testName + ".html"],
+                testname: testName === 'all' ? 'Unit Tests for Less.js' : testName,
+                browsers: browsers,
+                public: 'public',
+                recordVideo: false,
+                videoUploadOnPass: false,
+                recordScreenshots: process.env.TRAVIS_BRANCH !== "master",
+                build: process.env.TRAVIS_BRANCH === "master" ? process.env.TRAVIS_JOB_ID : undefined,
+                tags: [process.env.TRAVIS_BUILD_NUMBER, process.env.TRAVIS_PULL_REQUEST, process.env.TRAVIS_BRANCH],
+                statusCheckAttempts: -1,
+                sauceConfig: {
+                    'idle-timeout': 100
+                },
+                throttled: 5,
+                onTestComplete: function(result, callback) {
+                    // Called after a unit test is done, per page, per browser
+                    // 'result' param is the object returned by the test framework's reporter
+                    // 'callback' is a Node.js style callback function. You must invoke it after you
+                    // finish your work.
+                    // Pass a non-null value as the callback's first parameter if you want to throw an
+                    // exception. If your function is synchronous you can also throw exceptions
+                    // directly.
+                    // Passing true or false as the callback's second parameter passes or fails the
+                    // test. Passing undefined does not alter the test result. Please note that this
+                    // only affects the grunt task's result. You have to explicitly update the Sauce
+                    // Labs job's status via its REST API, if you want so.
+
+                    // This should be the encrypted value in Travis
+                    var user = process.env.SAUCE_USERNAME;
+                    var pass = process.env.SAUCE_ACCESS_KEY;
+
+                    git.short(function(hash) {
+                        require('phin')({
+                            method: 'PUT',
+                            url: ['https://saucelabs.com/rest/v1', user, 'jobs', result.job_id].join('/'),
+                            auth: { user: user, pass: pass },
+                            data: {
+                                passed: result.passed,
+                                build: 'build-' + hash
+                            }
+                        }, function (error, response) {
+                            if (error) {
+                                console.log(error);
+                                callback(error);
+                            } else if (response.statusCode !== 200) {
+                                console.log(response);
+                                callback(new Error('Unexpected response status'));
+                            } else {
+                                callback(null, result.passed);
+                            }
+                        });
+                    });
+
+                }
+            }
+        };
+    }
+
+    // Make the SauceLabs jobs
+    (['all'].concat(browserTests)).map(makeJob);
 
     // Project configuration.
     grunt.initConfig({
@@ -29,13 +173,22 @@ module.exports = function (grunt) {
         },
 
         shell: {
-            options: {stdout: true, failOnError: true},
+            options: {
+                stdout: true,
+                failOnError: true,
+                execOptions: {
+                    maxBuffer: Infinity
+                }
+            },
             test: {
                 command: 'node test/index.js'
             },
             benchmark: {
                 command: 'node benchmark/index.js'
             },
+            plugin: {
+                command: 'node bin/lessc --clean-css="--s1 --advanced" test/less/lazy-eval.less tmp/lazy-eval.css'
+            },
             "sourcemap-test": {
                 command: [
                     'node bin/lessc --source-map=test/sourcemaps/maps/import-map.map test/less/import.less test/sourcemaps/import.css',
@@ -107,24 +260,16 @@ module.exports = function (grunt) {
             }
         },
 
-        jshint: {
-            options: {jshintrc: '.jshintrc'},
-            files: {
-                src: [
-                    'Gruntfile.js',
-                    'lib/less/**/*.js',
-                    'lib/less-node/**/*.js',
-                    'lib/less-browser/**/*.js',
-                    'lib/less-rhino/**/*.js',
-                    'bin/lessc'
-                ]
-            }
-        },
-
-        jscs: {
-            src: ["test/**/*.js", "lib/less*/**/*.js", "bin/lessc"],
+        eslint: {
+            target: ["Gruntfile.js",
+                "test/**/*.js",
+                "lib/less*/**/*.js",
+                "bin/lessc",
+                "!test/browser/jasmine-jsreporter.js",
+                "!test/less/errors/plugin/plugin-error.js"
+            ],
             options: {
-                config: ".jscsrc"
+                configFile: ".eslintrc.json"
             }
         },
 
@@ -145,7 +290,15 @@ module.exports = function (grunt) {
             },
             main: {
                 // src is used to build list of less files to compile
-                src: ['test/less/*.less', '!test/less/javascript.less', '!test/less/urls.less', '!test/less/empty.less'],
+                src: [
+                    'test/less/*.less',
+                    // Don't test NPM import, obviously
+                    '!test/less/plugin-module.less',
+                    '!test/less/import-module.less',
+                    '!test/less/javascript.less',
+                    '!test/less/urls.less',
+                    '!test/less/empty.less'
+                ],
                 options: {
                     helpers: 'test/browser/runner-main-options.js',
                     specs: 'test/browser/runner-main-spec.js',
@@ -186,7 +339,7 @@ module.exports = function (grunt) {
                 }
             },
             browser: {
-                src: ['test/browser/less/*.less'],
+                src: ['test/browser/less/*.less', 'test/browser/less/plugin/*.less'],
                 options: {
                     helpers: 'test/browser/runner-browser-options.js',
                     specs: 'test/browser/runner-browser-spec.js',
@@ -272,66 +425,12 @@ module.exports = function (grunt) {
                     specs: 'test/browser/runner-filemanagerPlugin.js',
                     outfile: 'tmp/browser/test-runner-filemanager-plugin.html'
                 }
-            }            
-        },
-
-        'saucelabs-jasmine': {
-            all: {
-                options: {
-                    urls: ["filemanager-plugin","visitor-plugin","pre-processor-plugin","post-processor-plugin","post-processor", "global-vars", "modify-vars", "production", "rootpath-relative",
-                           "rootpath", "relative-urls", "browser", "no-js-errors", "legacy", "strict-units"
-                    ].map(function(testName) {
-                        return "http://localhost:8081/tmp/browser/test-runner-" + testName + ".html";
-                    }),
-                    testname: 'Sauce Unit Test for less.js',
-                    browsers: [{
-                        browserName: "chrome",
-                        version: '',
-                        platform: 'Windows 8'
-                    },
-                    {
-                        browserName: "firefox",
-                        version: '33',
-                        platform: 'Linux'
-                    },
-                    {
-                        browserName: "iPad",
-                        version: '8.0',
-                        platform: 'OS X 10.9',
-                        'device-orientation': 'portrait'
-                    },
-                    {
-                        browserName: "internet explorer",
-                        version: '8',
-                        platform: 'Windows XP'
-                    },
-                    {
-                        browserName: "internet explorer",
-                        version: '9',
-                        platform: 'Windows 7'
-                    },
-                    {
-                        browserName: "internet explorer",
-                        version: '10',
-                        platform: 'Windows 7'
-                    },
-                    {
-                        browserName: "internet explorer",
-                        version: '11',
-                        platform: 'Windows 8.1'
-                    }],
-                    sauceConfig: {
-                        'record-video': process.env.TRAVIS_BRANCH !== "master",
-                        'record-screenshots': process.env.TRAVIS_BRANCH !== "master",
-                        'idle-timeout': 100, 'max-duration': 120,
-                        build: process.env.TRAVIS_BRANCH === "master" ? process.env.TRAVIS_JOB_ID : undefined,
-                        tags: [process.env.TRAVIS_BUILD_NUMBER, process.env.TRAVIS_PULL_REQUEST, process.env.TRAVIS_BRANCH]
-                    },
-                    throttled: 3
-                }
             }
         },
 
+        'saucelabs-jasmine': sauceJobs,
+
+
         // Clean the version of less built for the tests
         clean: {
             test: ['test/browser/less.js', 'tmp', 'test/less-bom'],
@@ -341,6 +440,8 @@ module.exports = function (grunt) {
     });
 
     // Load these plugins to provide the necessary tasks
+    grunt.loadNpmTasks('grunt-saucelabs');
+
     require('jit-grunt')(grunt);
 
     // Actually load this plugin's task(s).
@@ -358,7 +459,7 @@ module.exports = function (grunt) {
         'uglify:dist'
     ]);
 
-    // Release Rhino Version
+    // Release Rhino Version (UNSUPPORTED)
     grunt.registerTask('rhino', [
         'browserify:rhino',
         'concat:rhino',
@@ -407,23 +508,27 @@ module.exports = function (grunt) {
         'sauce-after-setup'
     ]);
 
-    // setup a web server to run the browser tests in a browser rather than phantom
+    // var sauceTests = [];
+    // browserTests.map(function(testName) {
+    //     sauceTests.push('saucelabs-jasmine:' + testName);
+    // });
+
     grunt.registerTask('sauce-after-setup', [
-        'saucelabs-jasmine',
+        'saucelabs-jasmine:all',
         'clean:sauce_log'
     ]);
 
     var testTasks = [
         'clean',
-        'jshint',
-        'jscs',
+        'eslint',
         'shell:test',
+        'shell:plugin',
         'browsertest'
     ];
 
     if (isNaN(Number(process.env.TRAVIS_PULL_REQUEST, 10)) &&
-        Number(process.env.TRAVIS_NODE_VERSION) === 0.11 &&
-        (process.env.TRAVIS_BRANCH === "master" || process.env.TRAVIS_BRANCH === "sauce")) {
+        Number(process.env.TRAVIS_NODE_VERSION) === 4 &&
+        (process.env.TRAVIS_BRANCH === "master" || process.env.TRAVIS_BRANCH === "3.x")) {
         testTasks.push("force:on");
         testTasks.push("sauce-after-setup");
         testTasks.push("force:off");
@@ -433,7 +538,7 @@ module.exports = function (grunt) {
     grunt.registerTask('test', testTasks);
 
     // Run all tests
-    grunt.registerTask('quicktest', testTasks.slice(0, testTasks.length -1));
+    grunt.registerTask('quicktest', testTasks.slice(0, -1));
 
     // generate a good test environment for testing sourcemaps
     grunt.registerTask('sourcemap-test', [

package/README.md

@@ -1,6 +1,6 @@
-[![npm version](https://badge.fury.io/js/less.svg)](http://badge.fury.io/js/less) [![Build Status](https://travis-ci.org/less/less.js.svg?branch=master)](https://travis-ci.org/less/less.js)
-[![Dependencies](https://david-dm.org/less/less.js.svg)](https://david-dm.org/less/less.js) [![devDependency Status](https://david-dm.org/less/less.js/dev-status.svg)](https://david-dm.org/less/less.js#info=devDependencies) [![optionalDependency Status](https://david-dm.org/less/less.js/optional-status.svg)](https://david-dm.org/less/less.js#info=optionalDependencies)
-[![Sauce Test Status](https://saucelabs.com/browser-matrix/less.svg)](https://saucelabs.com/u/less) [![Build status](https://ci.appveyor.com/api/projects/status/bx2qspy3qbuxpl9q/branch/master?svg=true)](https://ci.appveyor.com/project/lukeapage/less-js/branch/master)
+[![npm version](https://badge.fury.io/js/less.svg)](http://badge.fury.io/js/less) [![Build Status](https://travis-ci.org/less/less.js.svg?branch=master)](https://travis-ci.org/less/less.js) [![Build status](https://ci.appveyor.com/api/projects/status/bx2qspy3qbuxpl9q/branch/master?svg=true)](https://ci.appveyor.com/project/lukeapage/less-js/branch/master) [![Dependencies](https://david-dm.org/less/less.js.svg)](https://david-dm.org/less/less.js) [![devDependency Status](https://david-dm.org/less/less.js/dev-status.svg)](https://david-dm.org/less/less.js#info=devDependencies) [![optionalDependency Status](https://david-dm.org/less/less.js/optional-status.svg)](https://david-dm.org/less/less.js#info=optionalDependencies) [![Twitter Follow](https://img.shields.io/twitter/follow/lesstocss.svg?style=flat-square)](https://twitter.com/lesstocss) [![Join the chat at https://gitter.im/less/less.js](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/less/less.js?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge) <sup>Chat with Less.js users</sup>
+
+[![Sauce Test Status](https://saucelabs.com/browser-matrix/less.svg)](https://saucelabs.com/u/less)
 
 # [Less.js](http://lesscss.org)
 
@@ -8,8 +8,6 @@
 
 This is the JavaScript, official, stable version of Less.
 
-###### :point_right: [![Join the chat at https://gitter.im/less/less.js](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/less/less.js?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge) <sup>Chat with Less.js users</sup>
-
 
 ## Getting Started
 
@@ -47,10 +45,10 @@ See the [changelog](CHANGELOG.md)
 
 ## [License](LICENSE)
 
-Copyright (c) 2009-2016 [Alexis Sellier](http://cloudhead.io) & The Core Less Team
+Copyright (c) 2009-2017 [Alexis Sellier](http://cloudhead.io) & The Core Less Team
 Licensed under the [Apache License](LICENSE).
 
 
-[so]: http://stackoverflow.com/questions/tagged/twitter-bootstrap+less "StackOverflow.com"
+[so]: http://stackoverflow.com/questions/tagged/less "StackOverflow.com"
 [issues]: https://github.com/less/less.js/issues "GitHub Issues for Less.js"
 [download]: https://github.com/less/less.js/zipball/master "Download Less.js"

package/appveyor.yml

@@ -1,21 +1,20 @@
 # Test against these versions of Node.js.
 environment:
   matrix:
-    - nodejs_version: "0.10"
-    - nodejs_version: "0.12"
     - nodejs_version: "4"
     - nodejs_version: "6"
+    - nodejs_version: "8"
+    - nodejs_version: "9"
 
 # Install scripts. (runs after repo cloning)
 install:
-  # Get the latest stable version of Node 0.STABLE.latest
-  - ps: Install-Product node $env:nodejs_version
-  # Use npm v2
-  - npm -g install npm@2
-  - set PATH=%APPDATA%\npm;%PATH%
-  - npm -v
-  # Typical npm stuff.
-  - npm install
+  # node 0.10 & 0.12 have race condition issues when running custom install scripts
+  # this can cause phantomjs-prebuilt install script to fail with the error:
+  # <Cannot find module 'boom'>
+  # Seems related to: https://github.com/npm/npm/issues/8152
+  # using <appveyor_retry> solves this.
+  - appveyor-retry call npm install
+
   # Grunt-specific stuff.
   - npm install -g grunt-cli