Skip to content

Commit 8e981bc

Browse files
authored
Remove tslint (#15012)
1 parent 70045a7 commit 8e981bc

File tree

14 files changed

+20
-699
lines changed

14 files changed

+20
-699
lines changed

.eslintignore

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,6 @@ build/webpack/loaders/remarkLoader.js
2121
build/webpack/loaders/jsonloader.js
2222
build/webpack/loaders/externalizeDependencies.js
2323

24-
build/tslint-rules/messagesMustBeLocalizedRule.js
25-
build/tslint-rules/baseRuleWalker.js
26-
2724
src/test/mocks/mementos.ts
2825
src/test/mocks/process.ts
2926
src/test/mocks/moduleInstaller.ts

.github/workflows/insiders.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,9 @@ jobs:
7878
- name: Check dependencies
7979
run: npm run checkDependencies
8080

81-
- name: Run linting on TypeScript code
82-
run: npx tslint --project tsconfig.json
81+
# TODO: Uncomment once we are ready to rely on eslint.
82+
# - name: Run linting on TypeScript code
83+
# run: npm run lint
8384

8485
- name: Run prettier on TypeScript code
8586
run: npx prettier 'src/**/*.ts*' --check

.github/workflows/pr-check.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,9 @@ jobs:
7878
- name: Check dependencies
7979
run: npm run checkDependencies
8080

81-
- name: Run linting on TypeScript code
82-
run: npx tslint --project tsconfig.json
81+
# TODO: Uncomment once we are ready to rely on eslint.
82+
# - name: Run linting on TypeScript code
83+
# run: npm run lint
8384

8485
- name: Run prettier on TypeScript code
8586
run: npx prettier 'src/**/*.ts*' --check

.github/workflows/release.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,9 @@ jobs:
7878
- name: Check dependencies
7979
run: npm run checkDependencies
8080

81-
- name: Run linting on TypeScript code
82-
run: npx tslint --project tsconfig.json
81+
# TODO: Uncomment once we are ready to rely on eslint.
82+
# - name: Run linting on TypeScript code
83+
# run: npm run lint
8384

8485
### Non-smoke tests
8586
tests:

.vscode/extensions.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
// See https://go.microsoft.com/fwlink/?LinkId=827846
33
// for the documentation about the extensions.json format
44
"recommendations": [
5-
"ms-vscode.vscode-typescript-tslint-plugin",
65
"editorconfig.editorconfig",
76
"esbenp.prettier-vscode",
87
"dbaeumer.vscode-eslint"

.vscode/settings.json

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@
3131
"editor.formatOnSave": true
3232
},
3333
"typescript.tsdk": "./node_modules/typescript/lib", // we want to use the TS server from our node_modules folder to control its version
34-
"tslint.enable": true,
3534
"python.linting.enabled": false,
3635
"python.testing.promptToConfigure": false,
3736
"python.workspaceSymbols.enabled": false,
@@ -42,8 +41,7 @@
4241
"prettier.printWidth": 120,
4342
"prettier.singleQuote": true,
4443
"editor.codeActionsOnSave": {
45-
"source.fixAll.eslint": true,
46-
"source.fixAll.tslint": true
44+
"source.fixAll.eslint": true
4745
},
4846
"python.languageServer": "Pylance",
4947
"python.analysis.logLevel": "Trace",

.vscode/tasks.json

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,7 @@
1212
"type": "npm",
1313
"script": "compile",
1414
"isBackground": true,
15-
"problemMatcher": [
16-
"$tsc-watch",
17-
{
18-
"base": "$tslint5",
19-
"fileLocation": "relative"
20-
}
21-
],
15+
"problemMatcher": ["$tsc-watch"],
2216
"group": {
2317
"kind": "build",
2418
"isDefault": true
@@ -33,10 +27,7 @@
3327
"kind": "build",
3428
"isDefault": true
3529
},
36-
"problemMatcher": [
37-
"$tsc-watch",
38-
"$ts-checker-webpack-watch"
39-
]
30+
"problemMatcher": ["$tsc-watch"]
4031
},
4132
{
4233
"label": "Run Unit Tests",

.vscodeignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ sprint-planning.github-issues
2121
test.ipynb
2222
tsconfig*.json
2323
tsfmt.json
24-
tslint.json
2524
vscode-python-signing.*
2625

2726
.github/**

CONTRIBUTING.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ The following projects have their own repositories:
2323
1. Windows, macOS, or Linux
2424
1. [Visual Studio Code](https://code.visualstudio.com/)
2525
1. The following VS Code extensions:
26-
- [TSLint](https://marketplace.visualstudio.com/items?itemName=ms-vscode.vscode-typescript-tslint-plugin)
26+
- [ESlint](https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint)
2727
- [Prettier](https://marketplace.visualstudio.com/items?itemName=esbenp.prettier-vscode)
2828
- [EditorConfig for VS Code](https://marketplace.visualstudio.com/items?itemName=EditorConfig.EditorConfig)
2929
1. Have an issue which has a "needs PR" label (feel free to indicate you would like to provide a PR for the issue so others don't work on it as well)
@@ -49,7 +49,7 @@ If you see warnings that `The engine "vscode" appears to be invalid.`, you can i
4949

5050
### Incremental Build
5151

52-
Run the `Compile` and `Hygiene` build Tasks from the [Run Build Task...](https://code.visualstudio.com/docs/editor/tasks) command picker (short cut `CTRL+SHIFT+B` or `⇧⌘B`). This will leave build and hygiene tasks running in the background and which will re-run as files are edited and saved. You can see the output from either task in the Terminal panel (use the selector to choose which output to look at).
52+
Run the `Compile` build task from the [Run Build Task...](https://code.visualstudio.com/docs/editor/tasks) command picker (short cut `CTRL+SHIFT+B` or `⇧⌘B`). This will leave build task running in the background and which will re-run as files are edited and saved. You can see the output from either task in the Terminal panel (use the selector to choose which output to look at).
5353

5454
You can also compile from the command-line. For a full compile you can use:
5555

build/tslint-rules/baseRuleWalker.js

Lines changed: 0 additions & 20 deletions
This file was deleted.

0 commit comments

Comments
 (0)