Skip to content

Commit 9a160f2

Browse files
sheetalkamatsheetalkamat
committed
Merge branch 'master' into watchOptions
2 parents c5b21d4 + b00c13b commit 9a160f2

File tree

1,212 files changed

+15815
-4308
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

1,212 files changed

+15815
-4308
lines changed

‎.circleci/config.yml‎

Lines changed: 90 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,90 @@
1+
workflows:
2+
version: 2
3+
main:
4+
jobs:
5+
- node9:
6+
filters:
7+
branches:
8+
only:
9+
- master
10+
- release-2.5
11+
- release-2.6
12+
- release-2.7
13+
- node8:
14+
filters:
15+
branches:
16+
only:
17+
- master
18+
- release-2.5
19+
- release-2.6
20+
- release-2.7
21+
- node6:
22+
filters:
23+
branches:
24+
only:
25+
- master
26+
- release-2.5
27+
- release-2.6
28+
- release-2.7
29+
nightly:
30+
triggers:
31+
- schedule:
32+
cron: "0 8 * * *"
33+
filters:
34+
branches:
35+
only: master
36+
jobs:
37+
- node9:
38+
filters:
39+
branches:
40+
only:
41+
- master
42+
- release-2.5
43+
- release-2.6
44+
- release-2.7
45+
context: nightlies
46+
- node8:
47+
filters:
48+
branches:
49+
only:
50+
- master
51+
- release-2.5
52+
- release-2.6
53+
- release-2.7
54+
context: nightlies
55+
- node6:
56+
filters:
57+
branches:
58+
only:
59+
- master
60+
- release-2.5
61+
- release-2.6
62+
- release-2.7
63+
context: nightlies
64+
65+
base: &base
66+
environment:
67+
- workerCount: 4
68+
steps:
69+
- checkout
70+
- run: |
71+
npm uninstall typescript --no-save
72+
npm uninstall tslint --no-save
73+
npm install
74+
#npm update Appeared in Jenkins only
75+
npm test
76+
77+
version: 2
78+
jobs:
79+
node9:
80+
docker:
81+
- image: circleci/node:9
82+
<<: *base
83+
node8:
84+
docker:
85+
- image: circleci/node:8
86+
<<: *base
87+
node6:
88+
docker:
89+
- image: circleci/node:6
90+
<<: *base

‎Gulpfile.ts‎

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ const cmdLineOptions = minimist(process.argv.slice(2), {
5353
"ru": "runners", "runner": "runners",
5454
"r": "reporter",
5555
"c": "colors", "color": "colors",
56-
"f": "files", "file": "files",
5756
"w": "workers",
5857
},
5958
default: {
@@ -69,7 +68,6 @@ const cmdLineOptions = minimist(process.argv.slice(2), {
6968
light: process.env.light === undefined || process.env.light !== "false",
7069
reporter: process.env.reporter || process.env.r,
7170
lint: process.env.lint || true,
72-
files: process.env.f || process.env.file || process.env.files || "",
7371
workers: process.env.workerCount || os.cpus().length,
7472
}
7573
});
@@ -1112,13 +1110,11 @@ function spawnLintWorker(files: {path: string}[], callback: (failures: number) =
11121110

11131111
gulp.task("lint", "Runs tslint on the compiler sources. Optional arguments are: --f[iles]=regex", ["build-rules"], () => {
11141112
if (fold.isTravis()) console.log(fold.start("lint"));
1115-
const fileMatcher = cmdLineOptions.files;
1116-
const files = fileMatcher
1117-
? `src/**/${fileMatcher}`
1118-
: `Gulpfile.ts "scripts/generateLocalizedDiagnosticMessages.ts" "scripts/tslint/**/*.ts" "src/**/*.ts" --exclude "src/lib/*.d.ts"`;
1119-
const cmd = `node node_modules/tslint/bin/tslint ${files} --formatters-dir ./built/local/tslint/formatters --format autolinkableStylish`;
1120-
console.log("Linting: " + cmd);
1121-
child_process.execSync(cmd, { stdio: [0, 1, 2] });
1113+
for (const project of ["scripts/tslint/tsconfig.json", "src/tsconfig-base.json"]) {
1114+
const cmd = `node node_modules/tslint/bin/tslint --project ${project} --formatters-dir ./built/local/tslint/formatters --format autolinkableStylish`;
1115+
console.log("Linting: " + cmd);
1116+
child_process.execSync(cmd, { stdio: [0, 1, 2] });
1117+
}
11221118
if (fold.isTravis()) console.log(fold.end("lint"));
11231119
});
11241120

‎Jakefile.js‎

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,7 @@ var harnessSources = harnessCoreSources.concat([
141141
"typingsInstaller.ts",
142142
"projectErrors.ts",
143143
"matchFiles.ts",
144+
"organizeImports.ts",
144145
"initializeTSConfig.ts",
145146
"extractConstants.ts",
146147
"extractFunctions.ts",
@@ -1301,15 +1302,13 @@ function spawnLintWorker(files, callback) {
13011302
desc("Runs tslint on the compiler sources. Optional arguments are: f[iles]=regex");
13021303
task("lint", ["build-rules"], () => {
13031304
if (fold.isTravis()) console.log(fold.start("lint"));
1304-
const fileMatcher = process.env.f || process.env.file || process.env.files;
1305-
1306-
const files = fileMatcher
1307-
? `src/**/${fileMatcher}`
1308-
: `Gulpfile.ts scripts/generateLocalizedDiagnosticMessages.ts "scripts/tslint/**/*.ts" "src/**/*.ts" --exclude "src/lib/*.d.ts"`;
1309-
const cmd = `node node_modules/tslint/bin/tslint ${files} --formatters-dir ./built/local/tslint/formatters --format autolinkableStylish`;
1310-
console.log("Linting: " + cmd);
1311-
jake.exec([cmd], { interactive: true }, () => {
1305+
function lint(project, cb) {
1306+
const cmd = `node node_modules/tslint/bin/tslint --project ${project} --formatters-dir ./built/local/tslint/formatters --format autolinkableStylish`;
1307+
console.log("Linting: " + cmd);
1308+
jake.exec([cmd], { interactive: true, windowsVerbatimArguments: true }, cb);
1309+
}
1310+
lint("scripts/tslint/tsconfig.json", () => lint("src/tsconfig-base.json", () => {
13121311
if (fold.isTravis()) console.log(fold.end("lint"));
13131312
complete();
1314-
});
1313+
}));
13151314
});

‎lib/lib.es5.d.ts‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -792,8 +792,7 @@ interface Date {
792792

793793
interface DateConstructor {
794794
new(): Date;
795-
new(value: number): Date;
796-
new(value: string): Date;
795+
new(value: string | number): Date;
797796
new(year: number, month: number, date?: number, hours?: number, minutes?: number, seconds?: number, ms?: number): Date;
798797
(): string;
799798
readonly prototype: Date;

‎pull_request_template.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
Thank you for submitting a pull request!
33
44
Here's a checklist you might find useful.
5-
[ ] There is an associated issue that is labelled
5+
[ ] There is an associated issue that is labeled
66
'Bug' or 'help wanted' or is in the Community milestone
77
[ ] Code is up-to-date with the `master` branch
88
[ ] You've successfully run `jake runtests` locally

‎scripts/configurePrerelease.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ function updateTsFile(tsFilePath: string, tsFileContents: string, majorMinor: st
5555
const parsedMajorMinor = majorMinorMatch[1];
5656
ts.Debug.assert(parsedMajorMinor === majorMinor, "versionMajorMinor does not match.", () => `${tsFilePath}: '${parsedMajorMinor}'; package.json: '${majorMinor}'`);
5757

58-
const versionRgx = /export const version = `\$\{versionMajorMinor\}\.(\d)`;/;
58+
const versionRgx = /export const version = `\$\{versionMajorMinor\}\.(\d)(-dev)?`;/;
5959
const patchMatch = versionRgx.exec(tsFileContents);
6060
ts.Debug.assert(patchMatch !== null, "The file seems to no longer have a string matching", () => versionRgx.toString());
6161
const parsedPatch = patchMatch[1];
@@ -85,4 +85,4 @@ function getPrereleasePatch(tag: string, plainPatch: string): string {
8585
return `${plainPatch}-${tag}.${timeStr}`;
8686
}
8787

88-
main();
88+
main();

‎scripts/tslint/rules/booleanTriviaRule.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ function walk(ctx: Lint.WalkContext<void>): void {
2727
/** Skip certain function/method names whose parameter names are not informative. */
2828
function shouldIgnoreCalledExpression(expression: ts.Expression): boolean {
2929
if (expression.kind === ts.SyntaxKind.PropertyAccessExpression) {
30-
const methodName = (expression as ts.PropertyAccessExpression).name.text as string;
30+
const methodName = (expression as ts.PropertyAccessExpression).name.text;
3131
if (methodName.indexOf("set") === 0) {
3232
return true;
3333
}
@@ -45,7 +45,7 @@ function walk(ctx: Lint.WalkContext<void>): void {
4545
}
4646
}
4747
else if (expression.kind === ts.SyntaxKind.Identifier) {
48-
const functionName = (expression as ts.Identifier).text as string;
48+
const functionName = (expression as ts.Identifier).text;
4949
if (functionName.indexOf("set") === 0) {
5050
return true;
5151
}

‎scripts/tslint/rules/noUnnecessaryTypeAssertion2Rule.ts‎

Lines changed: 98 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,98 @@
1+
/**
2+
* @license
3+
* Copyright 2016 Palantir Technologies, Inc.
4+
*
5+
* Licensed under the Apache License, Version 2.0 (the "License");
6+
* you may not use this file except in compliance with the License.
7+
* You may obtain a copy of the License at
8+
*
9+
* http://www.apache.org/licenses/LICENSE-2.0
10+
*
11+
* Unless required by applicable law or agreed to in writing, software
12+
* distributed under the License is distributed on an "AS IS" BASIS,
13+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
14+
* See the License for the specific language governing permissions and
15+
* limitations under the License.
16+
*/
17+
18+
import * as ts from "typescript";
19+
import * as Lint from "tslint";
20+
21+
export class Rule extends Lint.Rules.TypedRule {
22+
/* tslint:disable:object-literal-sort-keys */
23+
public static metadata: Lint.IRuleMetadata = {
24+
ruleName: "no-unnecessary-type-assertion",
25+
description: "Warns if a type assertion does not change the type of an expression.",
26+
options: {
27+
type: "list",
28+
listType: {
29+
type: "array",
30+
items: { type: "string" },
31+
},
32+
},
33+
optionsDescription: "A list of whitelisted assertion types to ignore",
34+
type: "typescript",
35+
hasFix: true,
36+
typescriptOnly: true,
37+
requiresTypeInfo: true,
38+
};
39+
/* tslint:enable:object-literal-sort-keys */
40+
41+
public static FAILURE_STRING = "This assertion is unnecessary since it does not change the type of the expression.";
42+
43+
public applyWithProgram(sourceFile: ts.SourceFile, program: ts.Program): Lint.RuleFailure[] {
44+
return this.applyWithWalker(new Walker(sourceFile, this.ruleName, this.ruleArguments, program.getTypeChecker()));
45+
}
46+
}
47+
48+
class Walker extends Lint.AbstractWalker<string[]> {
49+
constructor(sourceFile: ts.SourceFile, ruleName: string, options: string[], private readonly checker: ts.TypeChecker) {
50+
super(sourceFile, ruleName, options);
51+
}
52+
53+
public walk(sourceFile: ts.SourceFile) {
54+
const cb = (node: ts.Node): void => {
55+
switch (node.kind) {
56+
case ts.SyntaxKind.TypeAssertionExpression:
57+
case ts.SyntaxKind.AsExpression:
58+
this.verifyCast(node as ts.TypeAssertion | ts.AsExpression);
59+
}
60+
61+
return ts.forEachChild(node, cb);
62+
};
63+
64+
return ts.forEachChild(sourceFile, cb);
65+
}
66+
67+
private verifyCast(node: ts.TypeAssertion | ts.NonNullExpression | ts.AsExpression) {
68+
if (ts.isAssertionExpression(node) && this.options.indexOf(node.type.getText(this.sourceFile)) !== -1) {
69+
return;
70+
}
71+
const castType = this.checker.getTypeAtLocation(node);
72+
if (castType === undefined) {
73+
return;
74+
}
75+
76+
if (node.kind !== ts.SyntaxKind.NonNullExpression &&
77+
(castType.flags & ts.TypeFlags.Literal ||
78+
castType.flags & ts.TypeFlags.Object &&
79+
(castType as ts.ObjectType).objectFlags & ts.ObjectFlags.Tuple) ||
80+
// Sometimes tuple types don't have ObjectFlags.Tuple set, like when
81+
// they're being matched against an inferred type. So, in addition,
82+
// check if any properties are numbers, which implies that this is
83+
// likely a tuple type.
84+
(castType.getProperties().some((symbol) => !isNaN(Number(symbol.name))))) {
85+
86+
// It's not always safe to remove a cast to a literal type or tuple
87+
// type, as those types are sometimes widened without the cast.
88+
return;
89+
}
90+
91+
const uncastType = this.checker.getTypeAtLocation(node.expression);
92+
if (uncastType === castType) {
93+
this.addFailureAtNode(node, Rule.FAILURE_STRING, node.kind === ts.SyntaxKind.TypeAssertionExpression
94+
? Lint.Replacement.deleteFromTo(node.getStart(), node.expression.getStart())
95+
: Lint.Replacement.deleteFromTo(node.expression.getEnd(), node.getEnd()));
96+
}
97+
}
98+
}

‎scripts/tslint/tsconfig.json‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
{
22
"compilerOptions": {
3+
"lib": ["es6"],
34
"noImplicitAny": true,
45
"noImplicitReturns": true,
56
"noImplicitThis": true,

0 commit comments

Comments
 (0)