Skip to content

Commit 5ce8c83

Browse files
a-tarasyuka-tarasyuk
committed
prefer-const
1 parent 504b932 commit 5ce8c83

File tree

16 files changed

+50
-53
lines changed

16 files changed

+50
-53
lines changed

‎.eslintrc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@
112112
"no-var": "off",
113113
"object-shorthand": "error",
114114
"one-var": "off",
115-
"prefer-const": "off",
115+
"prefer-const": "error",
116116
"prefer-object-spread": "error",
117117
"quote-props": ["error", "consistent-as-needed"],
118118
"quotes": ["error", "double", { "avoidEscape": true, "allowTemplateLiterals": true }],

‎scripts/authors.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ function getKnownAuthorMaps() {
7373
}
7474

7575
function deduplicate<T>(array: T[]): T[] {
76-
let result: T[] = [];
76+
const result: T[] = [];
7777
if (array) {
7878
for (const item of array) {
7979
if (result.indexOf(item) < 0) {

‎scripts/buildProtocol.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ class DeclarationsWalker {
4242
return;
4343
}
4444
this.visitedTypes.push(type);
45-
let s = type.aliasSymbol || type.getSymbol();
45+
const s = type.aliasSymbol || type.getSymbol();
4646
if (!s) {
4747
return;
4848
}
@@ -64,7 +64,7 @@ class DeclarationsWalker {
6464
}
6565
else {
6666
// splice declaration in final d.ts file
67-
let text = decl.getFullText();
67+
const text = decl.getFullText();
6868
this.text += `${text}\n`;
6969
// recursively pull all dependencies into result dts file
7070

‎scripts/errorCheck.ts‎

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,35 +1,35 @@
11
declare var require: any;
2-
let fs = require("fs");
3-
let async = require("async");
4-
let glob = require("glob");
2+
const fs = require("fs");
3+
const async = require("async");
4+
const glob = require("glob");
55

66
fs.readFile("src/compiler/diagnosticMessages.json", "utf-8", (err, data) => {
77
if (err) {
88
throw err;
99
}
1010

11-
let messages = JSON.parse(data);
12-
let keys = Object.keys(messages);
11+
const messages = JSON.parse(data);
12+
const keys = Object.keys(messages);
1313
console.log("Loaded " + keys.length + " errors");
1414

15-
for (let k of keys) {
15+
for (const k of keys) {
1616
messages[k].seen = false;
1717
}
1818

19-
let errRegex = /\(\d+,\d+\): error TS([^:]+):/g;
19+
const errRegex = /\(\d+,\d+\): error TS([^:]+):/g;
20+
const baseDir = "tests/baselines/reference/";
2021

21-
let baseDir = "tests/baselines/reference/";
2222
fs.readdir(baseDir, (err, files) => {
2323
files = files.filter(f => f.indexOf(".errors.txt") > 0);
24-
let tasks: Array<(callback: () => void) => void> = [];
24+
const tasks: Array<(callback: () => void) => void> = [];
2525
files.forEach(f => tasks.push(done => {
2626
fs.readFile(baseDir + f, "utf-8", (err, baseline) => {
2727
if (err) throw err;
2828

2929
let g: string[];
3030
while (g = errRegex.exec(baseline)) {
31-
var errCode = +g[1];
32-
let msg = keys.filter(k => messages[k].code === errCode)[0];
31+
const errCode = +g[1];
32+
const msg = keys.filter(k => messages[k].code === errCode)[0];
3333
messages[msg].seen = true;
3434
}
3535

@@ -40,7 +40,7 @@ fs.readFile("src/compiler/diagnosticMessages.json", "utf-8", (err, data) => {
4040
async.parallelLimit(tasks, 25, done => {
4141
console.log("== List of errors not present in baselines ==");
4242
let count = 0;
43-
for (let k of keys) {
43+
for (const k of keys) {
4444
if (messages[k].seen !== true) {
4545
console.log(k);
4646
count++;
@@ -52,8 +52,8 @@ fs.readFile("src/compiler/diagnosticMessages.json", "utf-8", (err, data) => {
5252
});
5353

5454
fs.readFile("src/compiler/diagnosticInformationMap.generated.ts", "utf-8", (err, data) => {
55-
let errorRegexp = /\s(\w+): \{ code/g;
56-
let errorNames: string[] = [];
55+
const errorRegexp = /\s(\w+): \{ code/g;
56+
const errorNames: string[] = [];
5757
let errMatch: string[];
5858
while (errMatch = errorRegexp.exec(data)) {
5959
errorNames.push(errMatch[1]);
@@ -62,20 +62,20 @@ fs.readFile("src/compiler/diagnosticInformationMap.generated.ts", "utf-8", (err,
6262
let allSrc = "";
6363
glob("./src/**/*.ts", {}, (err, files) => {
6464
console.log("Reading " + files.length + " source files");
65-
for (let file of files) {
65+
for (const file of files) {
6666
if (file.indexOf("diagnosticInformationMap.generated.ts") > 0) {
6767
continue;
6868
}
6969

70-
let src = fs.readFileSync(file, "utf-8");
70+
const src = fs.readFileSync(file, "utf-8");
7171
allSrc = allSrc + src;
7272
}
7373

7474
console.log("Consumed " + allSrc.length + " characters of source");
7575

7676
let count = 0;
7777
console.log("== List of errors not used in source ==");
78-
for (let errName of errorNames) {
78+
for (const errName of errorNames) {
7979
if (allSrc.indexOf(errName) < 0) {
8080
console.log(errName);
8181
count++;

‎scripts/importDefinitelyTypedTests/importDefinitelyTypedTests.ts‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ function filePathEndsWith(path: string, endingString: string): boolean {
4242
}
4343

4444
function copyFileSync(source: string, destination: string) {
45-
let text = fs.readFileSync(source);
45+
const text = fs.readFileSync(source);
4646
fs.writeFileSync(destination, text);
4747
}
4848

@@ -52,8 +52,8 @@ function importDefinitelyTypedTest(tscPath: string, rwcTestPath: string, testCas
5252
cmd += " @" + responseFile;
5353
}
5454

55-
let testDirectoryName = testCaseName + "_" + Math.floor((Math.random() * 10000) + 1);
56-
let testDirectoryPath = path.join(process.env.temp, testDirectoryName);
55+
const testDirectoryName = testCaseName + "_" + Math.floor((Math.random() * 10000) + 1);
56+
const testDirectoryPath = path.join(process.env.temp, testDirectoryName);
5757
if (fs.existsSync(testDirectoryPath)) {
5858
throw new Error("Could not create test directory");
5959
}
@@ -77,8 +77,8 @@ function importDefinitelyTypedTest(tscPath: string, rwcTestPath: string, testCas
7777
}
7878

7979
// copy generated file to output location
80-
let outputFilePath = path.join(testDirectoryPath, "iocapture0.json");
81-
let testCasePath = path.join(rwcTestPath, "DefinitelyTyped_" + testCaseName + ".json");
80+
const outputFilePath = path.join(testDirectoryPath, "iocapture0.json");
81+
const testCasePath = path.join(rwcTestPath, "DefinitelyTyped_" + testCaseName + ".json");
8282
copyFileSync(outputFilePath, testCasePath);
8383

8484
//console.log("output generated at: " + outputFilePath);
@@ -121,8 +121,8 @@ function importDefinitelyTypedTests(tscPath: string, rwcTestPath: string, defini
121121
throw err;
122122
}
123123

124-
let tsFiles: string[] = [];
125-
let testFiles: string[] = [];
124+
const tsFiles: string[] = [];
125+
const testFiles: string[] = [];
126126
let paramFile: string;
127127

128128
for (const filePath of files.map(f => path.join(directoryPath, f))) {

‎src/compiler/debug.ts‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
/* @internal */
22
namespace ts {
33
export namespace Debug {
4+
/* eslint-disable prefer-const */
45
export let currentAssertionLevel = AssertionLevel.None;
56
export let isDebugging = false;
7+
/* eslint-enable prefer-const */
68

79
export function shouldAssert(level: AssertionLevel): boolean {
810
return currentAssertionLevel >= level;
@@ -258,4 +260,4 @@ namespace ts {
258260
isDebugInfoEnabled = true;
259261
}
260262
}
261-
}
263+
}

‎src/compiler/program.ts‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -709,7 +709,6 @@ namespace ts {
709709
const { rootNames, options, configFileParsingDiagnostics, projectReferences } = createProgramOptions;
710710
let { oldProgram } = createProgramOptions;
711711

712-
let program: Program;
713712
let processingDefaultLibFiles: SourceFile[] | undefined;
714713
let processingOtherFiles: SourceFile[] | undefined;
715714
let files: SourceFile[];
@@ -905,7 +904,7 @@ namespace ts {
905904
// unconditionally set oldProgram to undefined to prevent it from being captured in closure
906905
oldProgram = undefined;
907906

908-
program = {
907+
const program: Program = {
909908
getRootFileNames: () => rootNames,
910909
getSourceFile,
911910
getSourceFileByPath,

‎src/compiler/sys.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -635,6 +635,7 @@ namespace ts {
635635
};
636636

637637
// TODO: GH#18217 this is used as if it's certainly defined in many places.
638+
// eslint-disable-next-line prefer-const
638639
export let sys: System = (() => {
639640
// NodeJS detects "\uFEFF" at the start of the string and *replaces* it with the actual
640641
// byte order mark from the specified encoding. Using any other byte order mark does

‎src/compiler/transformers/generators.ts‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1975,12 +1975,11 @@ namespace ts {
19751975
}
19761976

19771977
function cacheExpression(node: Expression): Identifier {
1978-
let temp: Identifier;
19791978
if (isGeneratedIdentifier(node) || getEmitFlags(node) & EmitFlags.HelperName) {
19801979
return <Identifier>node;
19811980
}
19821981

1983-
temp = createTempVariable(hoistVariableDeclaration);
1982+
const temp = createTempVariable(hoistVariableDeclaration);
19841983
emitAssignment(temp, node, /*location*/ node);
19851984
return temp;
19861985
}

‎src/compiler/utilities.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4724,7 +4724,7 @@ namespace ts {
47244724
return { span, newLength };
47254725
}
47264726

4727-
export let unchangedTextChangeRange = createTextChangeRange(createTextSpan(0, 0), 0);
4727+
export let unchangedTextChangeRange = createTextChangeRange(createTextSpan(0, 0), 0); // eslint-disable-line prefer-const
47284728

47294729
/**
47304730
* Called to merge all the changes that occurred across several versions of a script snapshot
@@ -7025,6 +7025,7 @@ namespace ts {
70257025
this.skipTrivia = skipTrivia || (pos => pos);
70267026
}
70277027

7028+
// eslint-disable-next-line prefer-const
70287029
export let objectAllocator: ObjectAllocator = {
70297030
getNodeConstructor: () => <any>Node,
70307031
getTokenConstructor: () => <any>Node,

0 commit comments

Comments
 (0)