Remove obsolete flags, stop terminating debug processes (#3016)

For #80

Author: DonJayamanne

.vscode/launch.json

@@ -126,7 +126,10 @@
  "sourceMaps": true,
  "outFiles": [
  "${workspaceFolder}/out/**/*.js"
- ]
+ ],
+ "env": {
+ "MOCHA_REPORTER_JUNIT": "true"
+ }
  },
  {
  "name": "Unit Tests (without VS Code, *.unit.test.ts)",

build/constants.js

@@ -8,6 +8,6 @@ exports.ExtensionRootDir = path.join(__dirname, '..');
 const jsonFileWithListOfOldFiles = path.join(__dirname, 'existingFiles.json');
 function getListOfExcludedFiles() {
  const files = JSON.parse(fs.readFileSync(jsonFileWithListOfOldFiles).toString());
- return files.map(file => path.join(exports.ExtensionRootDir, file));
+ return files.map(file => path.join(exports.ExtensionRootDir, file.replace(/\//g, path.sep)));
 }
 exports.filesNotToCheck = getListOfExcludedFiles();

build/constants.ts

@@ -11,7 +11,7 @@ export const ExtensionRootDir = path.join(__dirname, '..');
 const jsonFileWithListOfOldFiles = path.join(__dirname, 'existingFiles.json');
 function getListOfExcludedFiles() {
  const files = JSON.parse(fs.readFileSync(jsonFileWithListOfOldFiles).toString()) as string[];
- return files.map(file => path.join(ExtensionRootDir, file));
+ return files.map(file => path.join(ExtensionRootDir, file.replace(/\//g, path.sep)));
 }
 
 export const filesNotToCheck: string[] = getListOfExcludedFiles();

build/tslint-rules/baseRuleWalker.js

@@ -2,6 +2,7 @@
 // Licensed under the MIT License.
 'use strict';
 Object.defineProperty(exports, "__esModule", { value: true });
+const path = require("path");
 const Lint = require("tslint");
 const constants_1 = require("../constants");
 class BaseRuleWalker extends Lint.RuleWalker {
@@ -11,7 +12,7 @@ class BaseRuleWalker extends Lint.RuleWalker {
  }
  sholdIgnoreCcurrentFile(node) {
  const sourceFile = node.getSourceFile();
- return sourceFile && sourceFile.fileName && this.filesToIgnore.indexOf(sourceFile.fileName) >= 0;
+ return sourceFile && sourceFile.fileName && this.filesToIgnore.indexOf(sourceFile.fileName.replace(/\//g, path.sep)) >= 0;
  }
 }
 exports.BaseRuleWalker = BaseRuleWalker;

build/tslint-rules/baseRuleWalker.ts

@@ -3,6 +3,7 @@
 
 'use strict';
 
+import * as path from 'path';
 import * as Lint from 'tslint';
 import * as ts from 'typescript';
 import { filesNotToCheck } from '../constants';
@@ -11,6 +12,6 @@ export class BaseRuleWalker extends Lint.RuleWalker {
  private readonly filesToIgnore = filesNotToCheck;
  protected sholdIgnoreCcurrentFile(node: ts.Node) {
  const sourceFile = node.getSourceFile();
- return sourceFile && sourceFile.fileName && this.filesToIgnore.indexOf(sourceFile.fileName) >= 0;
+ return sourceFile && sourceFile.fileName && this.filesToIgnore.indexOf(sourceFile.fileName.replace(/\//g, path.sep)) >= 0;
  }
 }

build/tslint-rules/messagesMustBeLocalizedRule.js

@@ -12,7 +12,7 @@ const methodNames = [
  // From IOutputChannel (vscode.OutputChannel)
  'appendLine', 'appendLine'
 ];
-const failureMessage = 'Messages must be localized in the Python Extension';
+const failureMessage = 'Messages must be localized in the Python Extension (use src/client/common/utils/localize.ts)';
 class NoStringLiteralsInMessages extends baseRuleWalker_1.BaseRuleWalker {
  visitCallExpression(node) {
  const prop = node.expression;

gulpfile.js

@@ -84,7 +84,7 @@ const copyrightHeaders = [copyrightHeader.join('\n'), copyrightHeader.join('\r\n
 
 gulp.task('precommit', (done) => run({ exitOnError: true, mode: 'staged' }, done));
 
-gulp.task('hygiene-watch', () => gulp.watch(tsFilter, debounce(() => run({ mode: 'changes', skipFormatCheck: true, skipIndentationCheck: true, skipCopyrightCheck: true }), 100)));
+gulp.task('hygiene-watch', () => gulp.watch(tsFilter, gulp.series('hygiene-modified')));
 
 gulp.task('hygiene', (done) => run({ mode: 'all', skipFormatCheck: true, skipIndentationCheck: true }, done));
 
@@ -533,15 +533,15 @@ function getModifiedFilesSync() {
  .split(/\r?\n/)
  .filter(l => !!l)
  .filter(l => l.length > 0)
- .map(l => l.trim())
+ .map(l => l.trim().replace(/\//g, path.sep))
  .map(l => path.join(__dirname, l));
  } else {
  const out = cp.execSync('git status -u -s', { encoding: 'utf8' });
  return out
  .split(/\r?\n/)
  .filter(l => !!l)
  .filter(l => _.intersection(['M', 'A', 'R', 'C', 'U', '?'], l.substring(0, 2).trim().split('')).length > 0)
- .map(l => path.join(__dirname, l.substring(2).trim()));
+ .map(l => path.join(__dirname, l.substring(2).trim().replace(/\//g, path.sep)));
  }
 }
 
@@ -562,11 +562,13 @@ function getFileListToProcess(options) {
 
  // If we need only modified files, then filter the glob.
  if (options && options.mode === 'changes') {
- return getModifiedFilesSync();
+ return getModifiedFilesSync()
+ .filter(file => fs.existsSync(file));
  }
 
  if (options && options.mode === 'staged') {
- return getStagedFilesSync();
+ return getStagedFilesSync()
+ .filter(file => fs.existsSync(file));
  }
 
  return all;

src/client/debugger/debugAdapter/DebugClients/localDebugClientV2.ts

@@ -4,7 +4,7 @@
 'use strict';
 
 import { DebugSession } from 'vscode-debugadapter';
-import { IKnownLaunchRequestArguments, LaunchRequestArguments } from '../../types';
+import { LaunchRequestArguments } from '../../types';
 import { IDebugLauncherScriptProvider } from '../types';
 import { LocalDebugClient } from './LocalDebugClient';
 
@@ -18,11 +18,6 @@ export class LocalDebugClientV2 extends LocalDebugClient {
  if (this.args.noDebug) {
  additionalPtvsdArgs.push('--nodebug');
  }
- const multiProcessPropety: keyof IKnownLaunchRequestArguments = 'multiProcess';
- const multiProcess = (this.args as Object).hasOwnProperty(multiProcessPropety) ? this.args.multiProcess : true;
- if (multiProcess) {
- additionalPtvsdArgs.push('--multiprocess');
- }
  return [launcher, ...additionalPtvsdArgs, '--client', '--host', 'localhost', '--port', debugPort.toString()];
  }
  protected buildStandardArguments() {