Chore: Convert to ES6 that is supported on Node 4, commonjs modules and remove Babel (#14)

Author: corbinu

.babelrc

@@ -1,17 +1,8 @@
-sudo: false
 language: node_js
 node_js:
-  - "0.10"
-  - "0.11"
-  - "0.12"
-  - "iojs-1"
-  - "iojs-2"
-  - "iojs-3"
-  - "4"
-matrix:
-  allow_failures:
-    - node_js:
-      - "0.11"
-      - "iojs-1"
-      - "iojs-2"
-      - "iojs-3"
+    - "4"
+    - "5"
+    - "6"
+    - "7"
+sudo: false
+script: "npm test"

.travis.yml

@@ -26,7 +26,6 @@
 
 var gulp = require('gulp'),
     mocha = require('gulp-mocha'),
-    babel = require('gulp-babel'),
     git = require('gulp-git'),
     bump = require('gulp-bump'),
     filter = require('gulp-filter'),
@@ -38,10 +37,6 @@ var gulp = require('gulp'),
     eslint = require('gulp-eslint'),
     fs = require('fs');
 
-require('babel-register')({
-    only: /eslint-scope\/(src|test)\//
-});
-
 var TEST = [ 'test/*.js' ];
 var SOURCE = [ 'src/**/*.js' ];
 
@@ -68,34 +63,14 @@ var ESLINT_OPTION = {
     }
 };
 
-var BABEL_OPTIONS = JSON.parse(fs.readFileSync('.babelrc', { encoding: 'utf8' }));
-
-var build = lazypipe()
-    .pipe(sourcemaps.init)
-    .pipe(babel, BABEL_OPTIONS)
-    .pipe(sourcemaps.write)
-    .pipe(gulp.dest, 'lib');
-
-gulp.task('build-for-watch', function () {
-    return gulp.src(SOURCE).pipe(plumber()).pipe(build());
-});
-
-gulp.task('build', function () {
-    return gulp.src(SOURCE).pipe(build());
-});
-
-gulp.task('test', [ 'lint', 'build' ], function () {
+gulp.task('test', function () {
     return gulp.src(TEST)
         .pipe(mocha({
             reporter: 'spec',
             timeout: 100000 // 100s
         }));
 });
 
-gulp.task('watch', [ 'build-for-watch' ], function () {
-    gulp.watch(SOURCE, [ 'build-for-watch' ]);
-});
-
 // Currently, not works for ES6.
 gulp.task('lint', function () {
     return gulp.src(SOURCE)
@@ -135,9 +110,9 @@ function inc(importance) {
         }));
 }
 
-gulp.task('patch', [ 'build' ], function () { return inc('patch'); })
-gulp.task('minor', [ 'build' ], function () { return inc('minor'); })
-gulp.task('major', [ 'build' ], function () { return inc('major'); })
+gulp.task('patch', function () { return inc('patch'); })
+gulp.task('minor', function () { return inc('minor'); })
+gulp.task('major', function () { return inc('major'); })
 
 gulp.task('travis', [ 'test' ]);
 gulp.task('default', [ 'travis' ]);

gulpfile.js

@@ -2,7 +2,7 @@
   "name": "eslint-scope",
   "description": "ECMAScript scope analyzer for ESLint",
   "homepage": "http://github.com/eslint/eslint-scope",
-  "main": "lib/index.js",
+  "main": "src/index.js",
   "version": "3.6.0",
   "engines": {
     "node": ">=0.4.0"
@@ -29,15 +29,11 @@
     "estraverse": "^4.1.1"
   },
   "devDependencies": {
-    "babel": "^6.3.26",
-    "babel-preset-es2015": "^6.3.13",
-    "babel-register": "^6.3.13",
     "chai": "^3.4.1",
     "eslint-release": "^0.10.1",
     "espree": "^3.1.1",
     "esprima": "^2.7.1",
     "gulp": "^3.9.0",
-    "gulp-babel": "^6.1.1",
     "gulp-bump": "^1.0.0",
     "gulp-eslint": "^1.1.1",
     "gulp-espower": "^1.0.2",
@@ -49,6 +45,7 @@
     "gulp-tag-version": "^1.3.0",
     "lazypipe": "^1.0.1",
     "vinyl-source-stream": "^1.1.0",
+    "typescript": "~2.0.10",
     "typescript-eslint-parser": "^1.0.0"
   }
 }

package.json

@@ -21,13 +21,14 @@
   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
   THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
+"use strict";
 
-import Variable from './variable';
+const Variable = require('./variable');
 
 /**
  * @class Definition
  */
-export default class Definition {
+class Definition {
     constructor(type, name, node, parent, index, kind) {
         /**
          * @member {String} Definition#type - type of the occurrence (e.g. "Parameter", "Variable", ...).
@@ -70,9 +71,9 @@ class ParameterDefinition extends Definition {
     }
 }
 
-export {
+module.exports = {
     ParameterDefinition,
     Definition
-}
+};
 
 /* vim: set sw=4 ts=4 et tw=80 : */

src/definition.js

@@ -45,17 +45,18 @@
  * The main interface is the {@link analyze} function.
  * @module escope
  */
+"use strict";
 
 /*jslint bitwise:true */
 
-import assert from 'assert';
+const assert = require('assert');
 
-import ScopeManager from './scope-manager';
-import Referencer from './referencer';
-import Reference from './reference';
-import Variable from './variable';
-import Scope from './scope';
-import { version } from '../package.json';
+const ScopeManager = require('./scope-manager');
+const Referencer = require('./referencer');
+const Reference = require('./reference');
+const Variable = require('./variable');
+const Scope = require('./scope');
+const version = require('../package.json').version;
 
 function defaultOptions() {
     return {
@@ -114,7 +115,7 @@ function updateDeeply(target, override) {
  * @param {string} [providedOptions.fallback='iteration'] - A kind of the fallback in order to encounter with unknown node. See [esrecurse](https://github.com/estools/esrecurse)'s the `fallback` option.
  * @return {ScopeManager}
  */
-export function analyze(tree, providedOptions) {
+function analyze(tree, providedOptions) {
     var scopeManager, referencer, options;
 
     options = updateDeeply(defaultOptions(), providedOptions);
@@ -129,7 +130,7 @@ export function analyze(tree, providedOptions) {
     return scopeManager;
 }
 
-export {
+module.exports = {
     /** @name module:escope.version */
     version,
     /** @name module:escope.Reference */
@@ -139,7 +140,8 @@ export {
     /** @name module:escope.Scope */
     Scope,
     /** @name module:escope.ScopeManager */
-    ScopeManager
+    ScopeManager,
+    analyze
 };
 
 

src/index.js

@@ -21,15 +21,16 @@
   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
   THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
+"use strict";
 
-import { Syntax } from 'estraverse';
-import esrecurse from 'esrecurse';
+const Syntax = require('estraverse').Syntax;
+const esrecurse = require('esrecurse');
 
 function getLast(xs) {
     return xs[xs.length - 1] || null;
 }
 
-export default class PatternVisitor extends esrecurse.Visitor {
+class PatternVisitor extends esrecurse.Visitor {
     static isPattern(node) {
         var nodeType = node.type;
         return (
@@ -131,4 +132,6 @@ export default class PatternVisitor extends esrecurse.Visitor {
     }
 }
 
+module.exports = PatternVisitor;
+
 /* vim: set sw=4 ts=4 et tw=80 : */

src/pattern-visitor.js

@@ -21,6 +21,7 @@
   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
   THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
+"use strict";
 
 const READ = 0x1;
 const WRITE = 0x2;
@@ -30,7 +31,7 @@ const RW = READ | WRITE;
  * A Reference represents a single occurrence of an identifier in code.
  * @class Reference
  */
-export default class Reference {
+class Reference {
     constructor(ident, scope, flag,  writeExpr, maybeImplicitGlobal, partial, init) {
         /**
          * Identifier syntax node.
@@ -151,4 +152,6 @@ Reference.WRITE = WRITE;
  */
 Reference.RW = RW;
 
+module.exports = Reference;
+
 /* vim: set sw=4 ts=4 et tw=80 : */

src/reference.js

@@ -21,13 +21,18 @@
   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
   THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
-import { Syntax } from 'estraverse';
-import esrecurse from 'esrecurse';
-import Reference from './reference';
-import Variable from './variable';
-import PatternVisitor from './pattern-visitor';
-import { ParameterDefinition, Definition } from './definition';
-import assert from 'assert';
+"use strict";
+
+const Syntax = require('estraverse').Syntax;
+const esrecurse = require('esrecurse');
+const Reference = require('./reference');
+const Variable = require('./variable');
+const PatternVisitor = require('./pattern-visitor');
+const definition = require('./definition');
+const assert = require('assert');
+
+const ParameterDefinition = definition.ParameterDefinition;
+const Definition = definition.Definition;
 
 function traverseIdentifierInPattern(options, rootPattern, referencer, callback) {
     // Call the callback at left hand identifier nodes, and Collect right hand nodes.
@@ -90,7 +95,7 @@ class Importer extends esrecurse.Visitor {
 }
 
 // Referencing variables and creating bindings.
-export default class Referencer extends esrecurse.Visitor {
+class Referencer extends esrecurse.Visitor {
     constructor(options, scopeManager) {
         super(null, options);
         this.options = options;
@@ -585,4 +590,6 @@ export default class Referencer extends esrecurse.Visitor {
     }
 }
 
+module.exports = Referencer;
+
 /* vim: set sw=4 ts=4 et tw=80 : */

src/referencer.js

@@ -21,29 +21,28 @@
   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
   THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
-
-import WeakMap from 'es6-weak-map';
-import Scope from './scope';
-import assert from 'assert';
-
-import {
-    GlobalScope,
-    CatchScope,
-    WithScope,
-    ModuleScope,
-    ClassScope,
-    SwitchScope,
-    FunctionScope,
-    ForScope,
-    TDZScope,
-    FunctionExpressionNameScope,
-    BlockScope
-} from './scope';
+"use strict";
+
+const WeakMap = require('es6-weak-map');
+const Scope = require('./scope');
+const assert = require('assert');
+
+const GlobalScope = Scope.GlobalScope;
+const CatchScope = Scope.CatchScope;
+const WithScope = Scope.WithScope;
+const ModuleScope = Scope.ModuleScope;
+const ClassScope = Scope.ClassScope;
+const SwitchScope = Scope.SwitchScope;
+const FunctionScope = Scope.FunctionScope;
+const ForScope = Scope.ForScope;
+const TDZScope = Scope.TDZScope;
+const FunctionExpressionNameScope = Scope.FunctionExpressionNameScope;
+const BlockScope = Scope.BlockScope;
 
 /**
  * @class ScopeManager
  */
-export default class ScopeManager {
+class ScopeManager {
     constructor(options) {
         this.scopes = [];
         this.globalScope = null;
@@ -242,4 +241,6 @@ export default class ScopeManager {
     }
 }
 
+module.exports = ScopeManager;
+
 /* vim: set sw=4 ts=4 et tw=80 : */