Skip to content

Remove old code #16877

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Sep 16, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 0 additions & 25 deletions .eslintignore
Original file line number Diff line number Diff line change
Expand Up @@ -191,14 +191,6 @@ src/test/application/misc/joinMailingListPrompt.unit.test.ts

src/test/performance/load.perf.test.ts

src/test/refactor/extension.refactor.extract.method.test.ts
src/test/refactor/extension.refactor.extract.var.test.ts

src/test/workspaceSymbols/provider.unit.test.ts
src/test/workspaceSymbols/common.ts
src/test/workspaceSymbols/main.unit.test.ts
src/test/workspaceSymbols/generator.unit.test.ts

src/client/interpreter/configuration/interpreterSelector/commands/base.ts
src/client/interpreter/configuration/interpreterSelector/commands/resetInterpreter.ts
src/client/interpreter/configuration/interpreterSelector/commands/setShebangInterpreter.ts
Expand All @@ -223,27 +215,16 @@ src/client/terminals/codeExecution/terminalCodeExecution.ts
src/client/terminals/codeExecution/codeExecutionManager.ts
src/client/terminals/codeExecution/djangoContext.ts

src/client/providers/objectDefinitionProvider.ts
src/client/providers/serviceRegistry.ts
src/client/providers/symbolProvider.ts
src/client/providers/completionSource.ts
src/client/providers/renameProvider.ts
src/client/providers/hoverProvider.ts
src/client/providers/itemInfoSource.ts
src/client/providers/formatProvider.ts
src/client/providers/importSortProvider.ts
src/client/providers/replProvider.ts
src/client/providers/codeActionProvider/main.ts
src/client/providers/codeActionProvider/launchJsonCodeActionProvider.ts
src/client/providers/codeActionProvider/pythonCodeActionProvider.ts
src/client/providers/docStringFoldingProvider.ts
src/client/providers/linterProvider.ts
src/client/providers/simpleRefactorProvider.ts
src/client/providers/completionProvider.ts
src/client/providers/definitionProvider.ts
src/client/providers/referenceProvider.ts
src/client/providers/terminalProvider.ts
src/client/providers/signatureProvider.ts

src/client/activation/languageServer/manager.ts
src/client/activation/languageServer/languageServerExtension.ts
Expand Down Expand Up @@ -464,9 +445,3 @@ src/client/application/diagnostics/commands/factory.ts
src/client/application/diagnostics/commands/execVSCCommand.ts
src/client/application/diagnostics/commands/launchBrowser.ts

src/client/refactor/proxy.ts
src/client/workspaceSymbols/main.ts
src/client/workspaceSymbols/generator.ts
src/client/workspaceSymbols/parser.ts
src/client/workspaceSymbols/provider.ts

35 changes: 3 additions & 32 deletions .github/test_plan.md
Original file line number Diff line number Diff line change
Expand Up @@ -141,36 +141,15 @@ SPAM='hello ${WHO}'
- [ ] Create a virtual environment
- [ ] Install `requests` into the virtual environment

#### Pylint/default linting

[Prompting to install Pylint is covered under `Environments` above]

For testing the disablement of the default linting rules for Pylint:

```ini
# pylintrc
[MESSAGES CONTROL]
enable=bad-names
```

```python3
# example.py
foo = 42 # Marked as a disallowed name.
```

- [ ] Installation via the prompt installs Pylint as appropriate
- [ ] Uses `--user` for system-install of Python
- [ ] Installs into a virtual environment environment directly
- [ ] Pylint works
- [ ] The existence of a `pylintrc` file turns off the default rules

#### Other linters
#### Linting

**Note**:

- You can use the `Run Linting` command to run a newly installed linter
- When the extension installs a new linter, it turns off all other linters

- [ ] pylint works
- [ ] `Select linter` lists the linter and installs it if necessary
- [ ] flake8 works
- [ ] `Select linter` lists the linter and installs it if necessary
- [ ] mypy works
Expand Down Expand Up @@ -223,14 +202,6 @@ def foo():pass
- [ ] yapf
- [ ] `"editor.formatOnType": true` works and has expected results

#### [Refactoring](https://code.visualstudio.com/docs/python/editing#_refactoring)

- [ ] [`Extract Variable`](https://code.visualstudio.com/docs/python/editing#_extract-variable) works
- [ ] You are prompted to install `rope` if it is not already available
- [ ] [`Extract method`](https://code.visualstudio.com/docs/python/editing#_extract-method) works
- [ ] You are prompted to install `rope` if it is not already available
- [ ] [`Sort Imports`](https://code.visualstudio.com/docs/python/editing#_sort-imports) works

### [Debugging](https://code.visualstudio.com/docs/python/debugging)

- [ ] [Configurations](https://code.visualstudio.com/docs/python/debugging#_debugging-specific-app-types) work (see [`package.json`](https://github.com/Microsoft/vscode-python/blob/main/package.json) and the `"configurationSnippets"` section for all of the possible configurations)
Expand Down
2 changes: 0 additions & 2 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
".vscode-test": false,
".vscode test": false,
"**/.mypy_cache/**": true,
"**/.ropeproject/**": true
},
"search.exclude": {
"out": true, // set this to false to include "out" folder in search results
Expand Down Expand Up @@ -45,7 +44,6 @@
"typescript.tsdk": "./node_modules/typescript/lib", // we want to use the TS server from our node_modules folder to control its version
"python.linting.enabled": false,
"python.testing.promptToConfigure": false,
"python.workspaceSymbols.enabled": false,
"python.formatting.provider": "black",
"typescript.preferences.quoteStyle": "single",
"javascript.preferences.quoteStyle": "single",
Expand Down
1 change: 0 additions & 1 deletion build/conda-functional-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ pycodestyle
pydocstyle
nose
pytest==4.6.9 # Last version of pytest with Python 2.7 support
rope
fastapi ; python_version>='3.6'
uvicorn ; python_version>='3.6'
flask
Expand Down
1 change: 0 additions & 1 deletion build/test-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ pycodestyle
pydocstyle
prospector ; python_version > '2.7'
pytest<6 ; python_version > '2.7' # Tests currently fail against pytest 6.
rope
flask
fastapi ; python_version > '2.7'
uvicorn ; python_version > '2.7'
Expand Down
1 change: 1 addition & 0 deletions news/3 Code Health/10440.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Remove support for `rope`. Refactoring now supported via language servers.
1 change: 1 addition & 0 deletions news/3 Code Health/13321.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Remove `pylintMinimalCheckers` setting. Syntax errors now reported via language servers.
1 change: 1 addition & 0 deletions news/3 Code Health/16063.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Remove `ctags` support. Workspace symbols now supported via language servers.
69 changes: 0 additions & 69 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@
"onCommand:python.viewOutput",
"onCommand:python.execSelectionInTerminal",
"onCommand:python.execSelectionInDjangoShell",
"onCommand:python.buildWorkspaceSymbols",
"onCommand:python.startREPL",
"onCommand:python.goToPythonObject",
"onCommand:python.reportIssue",
Expand Down Expand Up @@ -308,11 +307,6 @@
"command": "python.analysis.restartLanguageServer",
"title": "%python.command.python.analysis.restartLanguageServer.title%"
},
{
"category": "Python",
"command": "python.buildWorkspaceSymbols",
"title": "%python.command.python.buildWorkspaceSymbols.title%"
},
{
"category": "Python",
"command": "python.clearPersistentStorage",
Expand Down Expand Up @@ -380,16 +374,6 @@
"command": "python.launchTensorBoard",
"title": "%python.command.python.launchTensorBoard.title%"
},
{
"category": "Python Refactor",
"command": "python.refactorExtractMethod",
"title": "%python.command.python.refactorExtractMethod.title%"
},
{
"category": "Python Refactor",
"command": "python.refactorExtractVariable",
"title": "%python.command.python.refactorExtractVariable.title%"
},
{
"category": "Python",
"command": "python.refreshTensorBoard",
Expand Down Expand Up @@ -1275,47 +1259,6 @@
"description": "Path to folder with a list of Virtual Environments (e.g. ~/.pyenv, ~/Envs, ~/.virtualenvs).",
"scope": "machine",
"type": "string"
},
"python.workspaceSymbols.ctagsPath": {
"default": "ctags",
"description": "Fully qualified path to the ctags executable (else leave as ctags, assuming it is in current path).",
"scope": "resource",
"type": "string"
},
"python.workspaceSymbols.enabled": {
"default": false,
"description": "Set to 'true' to enable ctags to provide Workspace Symbols.",
"scope": "resource",
"type": "boolean"
},
"python.workspaceSymbols.exclusionPatterns": {
"default": [
"**/site-packages/**"
],
"description": "Pattern used to exclude files and folders from ctags See http://ctags.sourceforge.net/ctags.html.",
"items": {
"type": "string"
},
"scope": "resource",
"type": "array"
},
"python.workspaceSymbols.rebuildOnFileSave": {
"default": true,
"description": "Whether to re-build the tags file on when changes made to python files are saved.",
"scope": "resource",
"type": "boolean"
},
"python.workspaceSymbols.rebuildOnStart": {
"default": true,
"description": "Whether to re-build the tags file on start (defaults to true).",
"scope": "resource",
"type": "boolean"
},
"python.workspaceSymbols.tagFilePath": {
"default": "${workspaceFolder}/.vscode/tags",
"description": "Fully qualified path to tag file (exuberant ctag file), used to provide workspace symbols.",
"scope": "resource",
"type": "string"
}
},
"title": "Python",
Expand Down Expand Up @@ -1826,18 +1769,6 @@
"group": "Python",
"when": "editorFocus && editorLangId == python"
},
{
"command": "python.refactorExtractMethod",
"group": "Refactor",
"title": "Refactor: Extract Method",
"when": "editorHasSelection && editorLangId == python && !notebookEditorFocused"
},
{
"command": "python.refactorExtractVariable",
"group": "Refactor",
"title": "Refactor: Extract Variable",
"when": "editorHasSelection && editorLangId == python && !notebookEditorFocused"
},
{
"command": "python.sortImports",
"group": "Refactor",
Expand Down
3 changes: 0 additions & 3 deletions package.nls.de.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,8 @@
"python.command.python.sortImports.title": "Sortieren der Importe",
"python.command.python.startREPL.title": "Starten des REPL",
"python.command.python.createTerminal.title": "Terminal erstellen",
"python.command.python.buildWorkspaceSymbols.title": "Arbeitsplatz-Symbole erstellen",
"python.command.python.execInTerminal.title": "Python-Datei im Terminal ausführen",
"python.command.python.setInterpreter.title": "Interpreter auswählen",
"python.command.python.refactorExtractVariable.title": "Variable extrahieren",
"python.command.python.refactorExtractMethod.title": "Methode extrahieren",
"python.command.python.execSelectionInTerminal.title": "Selektion/Reihe in Python-Terminal ausführen",
"python.command.python.execSelectionInDjangoShell.title": "Selektion/Reihe in Django-Shell ausführen",
"python.command.python.goToPythonObject.title": "Gehe zu Python-Objekt",
Expand Down
3 changes: 0 additions & 3 deletions package.nls.es.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,8 @@
"python.command.python.sortImports.title": "Ordenar importaciones",
"python.command.python.startREPL.title": "Nuevo REPL",
"python.command.python.createTerminal.title": "Nueva terminal",
"python.command.python.buildWorkspaceSymbols.title": "Compilar símbolos del área de trabajo",
"python.command.python.execInTerminal.title": "Ejecutar archivo Python en la terminal",
"python.command.python.setInterpreter.title": "Seleccionar intérprete",
"python.command.python.refactorExtractVariable.title": "Extraer variable",
"python.command.python.refactorExtractMethod.title": "Extraer método",
"python.command.python.execSelectionInTerminal.title": "Ejecutar línea/selección en la terminal",
"python.command.python.execSelectionInDjangoShell.title": "Ejecutar línea/selección en el intérprete de Django",
"python.command.python.goToPythonObject.title": "Ir al objeto de Python",
Expand Down
3 changes: 0 additions & 3 deletions package.nls.fr.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,8 @@
"python.command.python.sortImports.title": "Trier les imports",
"python.command.python.startREPL.title": "Démarrer la console interactive",
"python.command.python.createTerminal.title": "Créer un terminal",
"python.command.python.buildWorkspaceSymbols.title": "Construire les symboles de l'espace de travail",
"python.command.python.execInTerminal.title": "Exécuter le script Python dans un terminal",
"python.command.python.setInterpreter.title": "Sélectionner l'interpreteur",
"python.command.python.refactorExtractVariable.title": "Extraire la variable",
"python.command.python.refactorExtractMethod.title": "Extraire la méthode",
"python.command.python.execSelectionInTerminal.title": "Exécuter la ligne/sélection dans un terminal Python",
"python.command.python.execSelectionInDjangoShell.title": "Exécuter la ligne/sélection dans un shell Django",
"python.command.python.goToPythonObject.title": "Se rendre à l'objet Python",
Expand Down
3 changes: 0 additions & 3 deletions package.nls.it.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,8 @@
"python.command.python.sortImports.title": "Ordina gli import",
"python.command.python.startREPL.title": "Apri nuova REPL",
"python.command.python.createTerminal.title": "Apri nuovo terminale",
"python.command.python.buildWorkspaceSymbols.title": "Compila simboli dello spazio di lavoro",
"python.command.python.execInTerminal.title": "Esegui file Python nel terminale",
"python.command.python.setInterpreter.title": "Seleziona interprete",
"python.command.python.refactorExtractVariable.title": "Estrai variable",
"python.command.python.refactorExtractMethod.title": "Estrai metodo",
"python.command.python.execSelectionInTerminal.title": "Esegui selezione/linea nel terminale di Python",
"python.command.python.execSelectionInDjangoShell.title": "Esegui selezione/linea nella shell Django",
"python.command.python.goToPythonObject.title": "Vai a oggetto Python",
Expand Down
3 changes: 0 additions & 3 deletions package.nls.ja.json
Original file line number Diff line number Diff line change
@@ -1,11 +1,8 @@
{
"python.command.python.sortImports.title": "import 文を並び替える",
"python.command.python.startREPL.title": "REPL を開始",
"python.command.python.buildWorkspaceSymbols.title": "ワークスペースのシンボルをビルド",
"python.command.python.execInTerminal.title": "ターミナルで Python ファイルを実行",
"python.command.python.setInterpreter.title": "インタープリターを選択",
"python.command.python.refactorExtractVariable.title": "変数を抽出",
"python.command.python.refactorExtractMethod.title": "メソッドを抽出",
"python.command.python.execSelectionInTerminal.title": "Python ターミナルで選択範囲/行を実行",
"python.command.python.execSelectionInDjangoShell.title": "Django シェルで選択範囲/行を実行",
"python.command.python.goToPythonObject.title": "Python オブジェクトに移動",
Expand Down
3 changes: 0 additions & 3 deletions package.nls.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,13 @@
"python.command.python.sortImports.title": "Sort Imports",
"python.command.python.startREPL.title": "Start REPL",
"python.command.python.createTerminal.title": "Create Terminal",
"python.command.python.buildWorkspaceSymbols.title": "Build Workspace Symbols",
"python.command.python.execInTerminal.title": "Run Python File in Terminal",
"python.command.python.debugInTerminal.title": "Debug Python File in Terminal",
"python.command.python.setInterpreter.title": "Select Interpreter",
"python.command.python.switchOffInsidersChannel.title": "Switch to Default Channel",
"python.command.python.switchToDailyChannel.title": "Switch to Insiders Daily Channel",
"python.command.python.switchToWeeklyChannel.title": "Switch to Insiders Weekly Channel",
"python.command.python.clearWorkspaceInterpreter.title": "Clear Workspace Interpreter Setting",
"python.command.python.refactorExtractVariable.title": "Extract Variable",
"python.command.python.refactorExtractMethod.title": "Extract Method",
"python.command.python.viewOutput.title": "Show Output",
"python.command.python.viewLanguageServerOutput.title": "Show Language Server Output",
"python.command.python.configureTests.title": "Configure Tests",
Expand Down
3 changes: 0 additions & 3 deletions package.nls.ko-kr.json
Original file line number Diff line number Diff line change
@@ -1,11 +1,8 @@
{
"python.command.python.sortImports.title": "Import문 정렬",
"python.command.python.startREPL.title": "REPL 시작",
"python.command.python.buildWorkspaceSymbols.title": "작업 영역 기호 빌드",
"python.command.python.execInTerminal.title": "터미널에서 Python 파일 실행",
"python.command.python.setInterpreter.title": "인터프리터 선택",
"python.command.python.refactorExtractVariable.title": "변수 추출",
"python.command.python.refactorExtractMethod.title": "메서드 추출",
"python.command.python.execSelectionInTerminal.title": "Python 터미널에서 선택 영역/줄 실행",
"python.command.python.execSelectionInDjangoShell.title": "Django 셸에서 선택 영역/줄 실행",
"python.command.python.goToPythonObject.title": " Python 객체로 이동",
Expand Down
3 changes: 0 additions & 3 deletions package.nls.nl.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,8 @@
"python.command.python.sortImports.title": "Import sorteren",
"python.command.python.startREPL.title": "REPL starten",
"python.command.python.createTerminal.title": "Terminal aanmaken",
"python.command.python.buildWorkspaceSymbols.title": "Werkruimte-symbolen aanmaken",
"python.command.python.execInTerminal.title": "Python-bestand in terminal uitvoeren",
"python.command.python.setInterpreter.title": "Interpreter selecteren",
"python.command.python.refactorExtractVariable.title": "Variabelen selecteren",
"python.command.python.refactorExtractMethod.title": "Methode selecteren",
"python.command.python.execSelectionInTerminal.title": "Selectie/rij in Python-terminal uitvoeren",
"python.command.python.execSelectionInDjangoShell.title": "Selectie/rij in Django-shell uitvoeren",
"python.command.python.goToPythonObject.title": "Naar Python-object gaan",
Expand Down
3 changes: 0 additions & 3 deletions package.nls.pl.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,8 @@
"python.command.python.sortImports.title": "Sortuj importy",
"python.command.python.startREPL.title": "Uruchom REPL",
"python.command.python.createTerminal.title": "Otwórz Terminal",
"python.command.python.buildWorkspaceSymbols.title": "Zbuduj symbole dla przestrzeni roboczej",
"python.command.python.execInTerminal.title": "Uruchom plik pythonowy w terminalu",
"python.command.python.setInterpreter.title": "Wybierz wersję interpretera",
"python.command.python.refactorExtractVariable.title": "Wyodrębnij zmienną",
"python.command.python.refactorExtractMethod.title": "Wyodrębnij metodę",
"python.command.python.viewOutput.title": "Pokaż wyniki",
"python.command.python.configureTests.title": "Konfiguruj testy jednostkowe",
"python.command.python.execSelectionInTerminal.title": "Uruchom zaznaczony obszar w interpreterze Pythona",
Expand Down
Loading