GeezerVison is showing github.com — Remove old code (#16877) · IceJinx33/vscode-python@c94b1f4
File treeExpand file treeCollapse file tree Open diff view settings Some content is hiddenLarge Commits have some content hidden by default. Use the searchbox below for content that may be hidden. Dismiss bannerExpand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: .eslintignore -25Lines changed: 0 additions & 25 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -191,14 +191,6 @@ src/test/application/misc/joinMailingListPrompt.unit.test.ts | 191 | 191 |
| 192 | 192 | src/test/performance/load.perf.test.ts | 193 | 193 |
| 194 | | -src/test/refactor/extension.refactor.extract.method.test.ts | 195 | | -src/test/refactor/extension.refactor.extract.var.test.ts | 196 | | - | 197 | | -src/test/workspaceSymbols/provider.unit.test.ts | 198 | | -src/test/workspaceSymbols/common.ts | 199 | | -src/test/workspaceSymbols/main.unit.test.ts | 200 | | -src/test/workspaceSymbols/generator.unit.test.ts | 201 | | - | 202 | 194 | src/client/interpreter/configuration/interpreterSelector/commands/base.ts | 203 | 195 | src/client/interpreter/configuration/interpreterSelector/commands/resetInterpreter.ts | 204 | 196 | src/client/interpreter/configuration/interpreterSelector/commands/setShebangInterpreter.ts | @@ -223,27 +215,16 @@ src/client/terminals/codeExecution/terminalCodeExecution.ts | 223 | 215 | src/client/terminals/codeExecution/codeExecutionManager.ts | 224 | 216 | src/client/terminals/codeExecution/djangoContext.ts | 225 | 217 |
| 226 | | -src/client/providers/objectDefinitionProvider.ts | 227 | 218 | src/client/providers/serviceRegistry.ts | 228 | | -src/client/providers/symbolProvider.ts | 229 | 219 | src/client/providers/completionSource.ts | 230 | | -src/client/providers/renameProvider.ts | 231 | | -src/client/providers/hoverProvider.ts | 232 | | -src/client/providers/itemInfoSource.ts | 233 | 220 | src/client/providers/formatProvider.ts | 234 | 221 | src/client/providers/importSortProvider.ts | 235 | 222 | src/client/providers/replProvider.ts | 236 | 223 | src/client/providers/codeActionProvider/main.ts | 237 | 224 | src/client/providers/codeActionProvider/launchJsonCodeActionProvider.ts | 238 | 225 | src/client/providers/codeActionProvider/pythonCodeActionProvider.ts | 239 | | -src/client/providers/docStringFoldingProvider.ts | 240 | 226 | src/client/providers/linterProvider.ts | 241 | | -src/client/providers/simpleRefactorProvider.ts | 242 | | -src/client/providers/completionProvider.ts | 243 | | -src/client/providers/definitionProvider.ts | 244 | | -src/client/providers/referenceProvider.ts | 245 | 227 | src/client/providers/terminalProvider.ts | 246 | | -src/client/providers/signatureProvider.ts | 247 | 228 |
| 248 | 229 | src/client/activation/languageServer/manager.ts | 249 | 230 | src/client/activation/languageServer/languageServerExtension.ts | @@ -464,9 +445,3 @@ src/client/application/diagnostics/commands/factory.ts | 464 | 445 | src/client/application/diagnostics/commands/execVSCCommand.ts | 465 | 446 | src/client/application/diagnostics/commands/launchBrowser.ts | 466 | 447 |
| 467 | | -src/client/refactor/proxy.ts | 468 | | -src/client/workspaceSymbols/main.ts | 469 | | -src/client/workspaceSymbols/generator.ts | 470 | | -src/client/workspaceSymbols/parser.ts | 471 | | -src/client/workspaceSymbols/provider.ts | 472 | | - |
Collapse file Copy file name to clipboardExpand all lines: .github/test_plan.md +3-32Lines changed: 3 additions & 32 deletions - Display the source diff
- Display the rich diff
| Original file line number | Diff line number | Diff line change |
|---|
@@ -141,36 +141,15 @@ SPAM='hello ${WHO}' | 141 | 141 | - [ ] Create a virtual environment | 142 | 142 | - [ ] Install `requests` into the virtual environment | 143 | 143 |
| 144 | | -#### Pylint/default linting | 145 | | - | 146 | | -[Prompting to install Pylint is covered under `Environments` above] | 147 | | - | 148 | | -For testing the disablement of the default linting rules for Pylint: | 149 | | - | 150 | | -```ini | 151 | | -# pylintrc | 152 | | -[MESSAGES CONTROL] | 153 | | -enable=bad-names | 154 | | -``` | 155 | | - | 156 | | -```python3 | 157 | | -# example.py | 158 | | -foo = 42 # Marked as a disallowed name. | 159 | | -``` | 160 | | - | 161 | | -- [ ] Installation via the prompt installs Pylint as appropriate | 162 | | - - [ ] Uses `--user` for system-install of Python | 163 | | - - [ ] Installs into a virtual environment environment directly | 164 | | -- [ ] Pylint works | 165 | | -- [ ] The existence of a `pylintrc` file turns off the default rules | 166 | | - | 167 | | -#### Other linters | | 144 | +#### Linting | 168 | 145 |
| 169 | 146 | **Note**: | 170 | 147 |
| 171 | 148 | - You can use the `Run Linting` command to run a newly installed linter | 172 | 149 | - When the extension installs a new linter, it turns off all other linters | 173 | 150 |
| | 151 | +- [ ] pylint works | | 152 | + - [ ] `Select linter` lists the linter and installs it if necessary | 174 | 153 | - [ ] flake8 works | 175 | 154 | - [ ] `Select linter` lists the linter and installs it if necessary | 176 | 155 | - [ ] mypy works | @@ -223,14 +202,6 @@ def foo():pass | 223 | 202 | - [ ] yapf | 224 | 203 | - [ ] `"editor.formatOnType": true` works and has expected results | 225 | 204 |
| 226 | | -#### [Refactoring](https://code.visualstudio.com/docs/python/editing#_refactoring) | 227 | | - | 228 | | -- [ ] [`Extract Variable`](https://code.visualstudio.com/docs/python/editing#_extract-variable) works | 229 | | - - [ ] You are prompted to install `rope` if it is not already available | 230 | | -- [ ] [`Extract method`](https://code.visualstudio.com/docs/python/editing#_extract-method) works | 231 | | - - [ ] You are prompted to install `rope` if it is not already available | 232 | | -- [ ] [`Sort Imports`](https://code.visualstudio.com/docs/python/editing#_sort-imports) works | 233 | | - | 234 | 205 | ### [Debugging](https://code.visualstudio.com/docs/python/debugging) | 235 | 206 |
| 236 | 207 | - [ ] [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) | |
Collapse file Copy file name to clipboardExpand all lines: .vscode/settings.json -2Lines changed: 0 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
| 12 | 12 | ".vscode-test": false, | 13 | 13 | ".vscode test": false, | 14 | 14 | "**/.mypy_cache/**": true, | 15 | | - "**/.ropeproject/**": true | 16 | 15 | }, | 17 | 16 | "search.exclude": { | 18 | 17 | "out": true, // set this to false to include "out" folder in search results | | 45 | 44 | "typescript.tsdk": "./node_modules/typescript/lib", // we want to use the TS server from our node_modules folder to control its version | 46 | 45 | "python.linting.enabled": false, | 47 | 46 | "python.testing.promptToConfigure": false, | 48 | | - "python.workspaceSymbols.enabled": false, | 49 | 47 | "python.formatting.provider": "black", | 50 | 48 | "typescript.preferences.quoteStyle": "single", | 51 | 49 | "javascript.preferences.quoteStyle": "single", | |
Collapse file Copy file name to clipboardExpand all lines: build/conda-functional-requirements.txt -1Lines changed: 0 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -11,7 +11,6 @@ pycodestyle | 11 | 11 | pydocstyle | 12 | 12 | nose | 13 | 13 | pytest==4.6.9 # Last version of pytest with Python 2.7 support | 14 | | -rope | 15 | 14 | fastapi ; python_version>='3.6' | 16 | 15 | uvicorn ; python_version>='3.6' | 17 | 16 | flask | |
Collapse file Copy file name to clipboardExpand all lines: build/test-requirements.txt -1Lines changed: 0 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -10,7 +10,6 @@ pycodestyle | 10 | 10 | pydocstyle | 11 | 11 | prospector ; python_version > '2.7' | 12 | 12 | pytest<6 ; python_version > '2.7' # Tests currently fail against pytest 6. | 13 | | -rope | 14 | 13 | flask | 15 | 14 | fastapi ; python_version > '2.7' | 16 | 15 | uvicorn ; python_version > '2.7' | |
Collapse file Copy file name to clipboard +1Lines changed: 1 addition & 0 deletions - Display the source diff
- Display the rich diff
| Original file line number | Diff line number | Diff line change |
|---|
| | 1 | +Remove support for `rope`. Refactoring now supported via language servers. |
Collapse file Copy file name to clipboard +1Lines changed: 1 addition & 0 deletions - Display the source diff
- Display the rich diff
| Original file line number | Diff line number | Diff line change |
|---|
| | 1 | +Remove `pylintMinimalCheckers` setting. Syntax errors now reported via language servers. |
Collapse file Copy file name to clipboard +1Lines changed: 1 addition & 0 deletions - Display the source diff
- Display the rich diff
| Original file line number | Diff line number | Diff line change |
|---|
| | 1 | +Remove `ctags` support. Workspace symbols now supported via language servers. |
Collapse file Copy file name to clipboardExpand all lines: package.json -69Lines changed: 0 additions & 69 deletions | Original file line number | Diff line number | Diff line change |
|---|
| 67 | 67 | "onCommand:python.viewOutput", | 68 | 68 | "onCommand:python.execSelectionInTerminal", | 69 | 69 | "onCommand:python.execSelectionInDjangoShell", | 70 | | - "onCommand:python.buildWorkspaceSymbols", | 71 | 70 | "onCommand:python.startREPL", | 72 | 71 | "onCommand:python.goToPythonObject", | 73 | 72 | "onCommand:python.reportIssue", | | 308 | 307 | "command": "python.analysis.restartLanguageServer", | 309 | 308 | "title": "%python.command.python.analysis.restartLanguageServer.title%" | 310 | 309 | }, | 311 | | - { | 312 | | - "category": "Python", | 313 | | - "command": "python.buildWorkspaceSymbols", | 314 | | - "title": "%python.command.python.buildWorkspaceSymbols.title%" | 315 | | - }, | 316 | 310 | { | 317 | 311 | "category": "Python", | 318 | 312 | "command": "python.clearPersistentStorage", | | 380 | 374 | "command": "python.launchTensorBoard", | 381 | 375 | "title": "%python.command.python.launchTensorBoard.title%" | 382 | 376 | }, | 383 | | - { | 384 | | - "category": "Python Refactor", | 385 | | - "command": "python.refactorExtractMethod", | 386 | | - "title": "%python.command.python.refactorExtractMethod.title%" | 387 | | - }, | 388 | | - { | 389 | | - "category": "Python Refactor", | 390 | | - "command": "python.refactorExtractVariable", | 391 | | - "title": "%python.command.python.refactorExtractVariable.title%" | 392 | | - }, | 393 | 377 | { | 394 | 378 | "category": "Python", | 395 | 379 | "command": "python.refreshTensorBoard", | | 1275 | 1259 | "description": "Path to folder with a list of Virtual Environments (e.g. ~/.pyenv, ~/Envs, ~/.virtualenvs).", | 1276 | 1260 | "scope": "machine", | 1277 | 1261 | "type": "string" | 1278 | | - }, | 1279 | | - "python.workspaceSymbols.ctagsPath": { | 1280 | | - "default": "ctags", | 1281 | | - "description": "Fully qualified path to the ctags executable (else leave as ctags, assuming it is in current path).", | 1282 | | - "scope": "resource", | 1283 | | - "type": "string" | 1284 | | - }, | 1285 | | - "python.workspaceSymbols.enabled": { | 1286 | | - "default": false, | 1287 | | - "description": "Set to 'true' to enable ctags to provide Workspace Symbols.", | 1288 | | - "scope": "resource", | 1289 | | - "type": "boolean" | 1290 | | - }, | 1291 | | - "python.workspaceSymbols.exclusionPatterns": { | 1292 | | - "default": [ | 1293 | | - "**/site-packages/**" | 1294 | | - ], | 1295 | | - "description": "Pattern used to exclude files and folders from ctags See http://ctags.sourceforge.net/ctags.html.", | 1296 | | - "items": { | 1297 | | - "type": "string" | 1298 | | - }, | 1299 | | - "scope": "resource", | 1300 | | - "type": "array" | 1301 | | - }, | 1302 | | - "python.workspaceSymbols.rebuildOnFileSave": { | 1303 | | - "default": true, | 1304 | | - "description": "Whether to re-build the tags file on when changes made to python files are saved.", | 1305 | | - "scope": "resource", | 1306 | | - "type": "boolean" | 1307 | | - }, | 1308 | | - "python.workspaceSymbols.rebuildOnStart": { | 1309 | | - "default": true, | 1310 | | - "description": "Whether to re-build the tags file on start (defaults to true).", | 1311 | | - "scope": "resource", | 1312 | | - "type": "boolean" | 1313 | | - }, | 1314 | | - "python.workspaceSymbols.tagFilePath": { | 1315 | | - "default": "${workspaceFolder}/.vscode/tags", | 1316 | | - "description": "Fully qualified path to tag file (exuberant ctag file), used to provide workspace symbols.", | 1317 | | - "scope": "resource", | 1318 | | - "type": "string" | 1319 | 1262 | } | 1320 | 1263 | }, | 1321 | 1264 | "title": "Python", | | 1826 | 1769 | "group": "Python", | 1827 | 1770 | "when": "editorFocus && editorLangId == python" | 1828 | 1771 | }, | 1829 | | - { | 1830 | | - "command": "python.refactorExtractMethod", | 1831 | | - "group": "Refactor", | 1832 | | - "title": "Refactor: Extract Method", | 1833 | | - "when": "editorHasSelection && editorLangId == python && !notebookEditorFocused" | 1834 | | - }, | 1835 | | - { | 1836 | | - "command": "python.refactorExtractVariable", | 1837 | | - "group": "Refactor", | 1838 | | - "title": "Refactor: Extract Variable", | 1839 | | - "when": "editorHasSelection && editorLangId == python && !notebookEditorFocused" | 1840 | | - }, | 1841 | 1772 | { | 1842 | 1773 | "command": "python.sortImports", | 1843 | 1774 | "group": "Refactor", | |
Collapse file Copy file name to clipboardExpand all lines: package.nls.de.json -3Lines changed: 0 additions & 3 deletions | Original file line number | Diff line number | Diff line change |
|---|
| 2 | 2 | "python.command.python.sortImports.title": "Sortieren der Importe", | 3 | 3 | "python.command.python.startREPL.title": "Starten des REPL", | 4 | 4 | "python.command.python.createTerminal.title": "Terminal erstellen", | 5 | | - "python.command.python.buildWorkspaceSymbols.title": "Arbeitsplatz-Symbole erstellen", | 6 | 5 | "python.command.python.execInTerminal.title": "Python-Datei im Terminal ausführen", | 7 | 6 | "python.command.python.setInterpreter.title": "Interpreter auswählen", | 8 | | - "python.command.python.refactorExtractVariable.title": "Variable extrahieren", | 9 | | - "python.command.python.refactorExtractMethod.title": "Methode extrahieren", | 10 | 7 | "python.command.python.execSelectionInTerminal.title": "Selektion/Reihe in Python-Terminal ausführen", | 11 | 8 | "python.command.python.execSelectionInDjangoShell.title": "Selektion/Reihe in Django-Shell ausführen", | 12 | 9 | "python.command.python.goToPythonObject.title": "Gehe zu Python-Objekt", | |
|