Skip to content

Update debug python button #138

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

Conversation

paulacamargo25
Copy link
Contributor

@paulacamargo25 paulacamargo25 commented Nov 21, 2023

Closed: #137

Screenshot 2023-11-21 at 5 08 11 PM

@paulacamargo25 paulacamargo25 added the feature-request Request for new features or functionality label Nov 21, 2023
@paulacamargo25 paulacamargo25 added this to the November 2023 milestone Nov 21, 2023
@paulacamargo25 paulacamargo25 self-assigned this Nov 21, 2023
@roblourens
Copy link
Member

What's the difference between "Debug Python File" and "Python Debugger: Debug Python File"?

@@ -5,13 +5,17 @@ import * as path from 'path';
import * as fs from 'fs-extra';
import { parse } from 'jsonc-parser';
import { DebugConfiguration, Uri, WorkspaceFolder } from 'vscode';
import { getWorkspaceFolder } from '../../../common/vscodeapi';
import { getConfiguration, getWorkspaceFolder } from '../../../common/vscodeapi';

export async function getConfigurationsForWorkspace(workspace: WorkspaceFolder): Promise<DebugConfiguration[]> {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you need to look up launch configs for?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to the proposal, we check if there is any configuration already created. If not, we show the user the prompt to create the launch.json

@paulacamargo25
Copy link
Contributor Author

What's the difference between "Debug Python File" and "Python Debugger: Debug Python File"?

The first one come from the Python Extension and the new one is for the Python Debugger. When we remove the debugger from the python debugger there will only be one.

@paulacamargo25 paulacamargo25 merged commit 4718b3b into microsoft:main Nov 22, 2023
@roblourens
Copy link
Member

I would suggest removing the "Python Debugger" prefix from this extension's commands, it seems redundant

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve Debug Python File in Editor Menu
2 participants