Skip to content
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

Show warning and allow user to turn off smart send for deprecated Python code #22353

Merged
merged 31 commits into from
Nov 18, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
faebdb6
detect user is on file with deprecated Python code
anthonykim1 Oct 26, 2023
14f9b64
send warning message if smart send deprecated py
anthonykim1 Oct 26, 2023
e1d203f
add setting
anthonykim1 Oct 26, 2023
c5ced54
switch repl and code order to normal
anthonykim1 Oct 26, 2023
bc74433
comply with linting error
anthonykim1 Oct 26, 2023
d0ad1fb
fix for test failing
anthonykim1 Nov 13, 2023
be70c44
comment out unused variables
anthonykim1 Nov 13, 2023
d0fd80d
try adding configService and new REPL setting
anthonykim1 Nov 13, 2023
59805ad
quit unused var warning
anthonykim1 Nov 13, 2023
d911459
add activeResourceService
anthonykim1 Nov 13, 2023
a29ecdd
clean up
anthonykim1 Nov 13, 2023
924f9e8
respect deprecated
anthonykim1 Nov 14, 2023
fe80e93
make a button that leed to change setting
anthonykim1 Nov 14, 2023
e23c504
Change message wording
anthonykim1 Nov 14, 2023
f6f8a4a
Change wording
anthonykim1 Nov 14, 2023
91ffe64
use commandManager rather than calling vs code
anthonykim1 Nov 15, 2023
93ab499
add commandManager
anthonykim1 Nov 15, 2023
972f144
remove systemVariable resolve
anthonykim1 Nov 15, 2023
05d4e7f
add warning message test
anthonykim1 Nov 15, 2023
4ebc502
fix typo
anthonykim1 Nov 16, 2023
9283d80
fix smoke test error
anthonykim1 Nov 16, 2023
4682545
button will actually change setting
anthonykim1 Nov 16, 2023
d3efb7c
Update description according to suggestion
anthonykim1 Nov 16, 2023
f4deb62
add traceInfo message
anthonykim1 Nov 16, 2023
4ba135f
Update src/client/terminals/codeExecution/terminalCodeExecution.ts
anthonykim1 Nov 17, 2023
1794700
touch up on warning message
anthonykim1 Nov 17, 2023
13eeb80
check condition on Disable button
anthonykim1 Nov 17, 2023
4fc0357
Update src/client/terminals/codeExecution/terminalCodeExecution.ts
anthonykim1 Nov 17, 2023
956ee20
put message and button in localize
anthonykim1 Nov 17, 2023
16021ee
lowercase enable
anthonykim1 Nov 18, 2023
7063648
enable for test
anthonykim1 Nov 18, 2023
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
Prev Previous commit
Next Next commit
use commandManager rather than calling vs code
  • Loading branch information
anthonykim1 committed Nov 15, 2023
commit 91ffe6449de94eb3ebf7e84c6c9cebd2675495b0
1 change: 1 addition & 0 deletions src/client/common/application/commands.ts
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,7 @@ export interface ICommandNameArgumentTypeMapping extends ICommandNameWithoutArgu
];
['workbench.action.files.openFolder']: [];
['workbench.action.openWorkspace']: [];
['workbench.action.openSettings']: [string];
['setContext']: [string, boolean] | ['python.vscode.channel', Channel];
['python.reloadVSCode']: [string];
['revealLine']: [{ lineNumber: number; at: 'top' | 'center' | 'bottom' }];
Expand Down
6 changes: 3 additions & 3 deletions src/client/terminals/codeExecution/terminalCodeExecution.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
import { inject, injectable } from 'inversify';
import * as path from 'path';
import { Disposable, l10n, Uri } from 'vscode';
import { IWorkspaceService } from '../../common/application/types';
import { ICommandManager, IWorkspaceService } from '../../common/application/types';
import { Commands } from '../../common/constants';
import '../../common/extensions';
import { IPlatformService } from '../../common/platform/types';
Expand All @@ -16,7 +16,6 @@ import { showWarningMessage } from '../../common/vscodeApis/windowApis';
import { IInterpreterService } from '../../interpreter/contracts';
import { buildPythonExecInfo, PythonExecInfo } from '../../pythonEnvironments/exec';
import { ICodeExecutionService } from '../../terminals/types';
import * as vscode from 'vscode';
@injectable()
export class TerminalCodeExecutionProvider implements ICodeExecutionService {
private hasRanOutsideCurrentDrive = false;
Expand All @@ -29,6 +28,7 @@ export class TerminalCodeExecutionProvider implements ICodeExecutionService {
@inject(IDisposableRegistry) protected readonly disposables: Disposable[],
@inject(IPlatformService) protected readonly platformService: IPlatformService,
@inject(IInterpreterService) protected readonly interpreterService: IInterpreterService,
@inject(ICommandManager) protected readonly commandManager: ICommandManager,
) {}

public async executeFile(file: Uri, options?: { newTerminalPerFile: boolean }) {
Expand Down Expand Up @@ -56,7 +56,7 @@ export class TerminalCodeExecutionProvider implements ICodeExecutionService {
'Switch to line-by-line',
anthonykim1 marked this conversation as resolved.
Show resolved Hide resolved
);
if (selection === 'Switch to line-by-line') {
vscode.commands.executeCommand('workbench.action.openSettings', 'python.REPL.EnableREPLSmartSend');
this.commandManager.executeCommand('workbench.action.openSettings', 'python.REPL.EnableREPLSmartSend');
}
} else {
await this.getTerminalService(resource).sendText(code);
Expand Down
Loading