make linc script cross platform (#11447)

* make linc script cross platform

* fix typo

* attempt to fix long command error

* use eslint node api

* Update linc.js

Author: xjlim

package.json

@@ -98,7 +98,7 @@
   },
   "scripts": {
     "build": "npm run version-check && node scripts/rollup/build.js",
-    "linc": "git diff --name-only --diff-filter=ACMRTUB `git merge-base HEAD master` | grep '\\.js$' | xargs eslint --",
+    "linc": "node ./scripts/tasks/linc.js",
     "lint": "node ./scripts/tasks/eslint.js",
     "postinstall": "node node_modules/fbjs-scripts/node/check-dev-engines.js package.json",
     "test": "jest",

scripts/tasks/linc.js

@@ -0,0 +1,41 @@
+/**
+ * Copyright (c) 2013-present, Facebook, Inc.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+'use strict';
+
+const execFileSync = require('child_process').execFileSync;
+const CLIEngine = require('eslint').CLIEngine;
+
+const cli = new CLIEngine();
+const formatter = cli.getFormatter();
+
+const mergeBase = execFileSync('git', ['merge-base', 'HEAD', 'master'], {
+  stdio: 'pipe',
+  encoding: 'utf-8',
+}).trim();
+const changedFiles = execFileSync(
+  'git',
+  ['diff', '--name-only', '--diff-filter=ACMRTUB', mergeBase],
+  {
+    stdio: 'pipe',
+    encoding: 'utf-8',
+  }
+)
+  .trim()
+  .toString()
+  .split('\n');
+const jsFiles = changedFiles.filter(file => file.match(/.js$/g));
+
+const report = cli.executeOnFiles(jsFiles);
+console.log(formatter(report.results));
+
+if (report.errorCount > 0) {
+  console.log('Lint failed for changed files.');
+  process.exit(1);
+} else {
+  console.log('Lint passed for changed files.');
+}