-
Notifications
You must be signed in to change notification settings - Fork 9
Adding menu items #310
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
Adding menu items #310
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some initial wording feedback
public readonly name = 'newScript'; | ||
public readonly displayName = 'Script'; | ||
public readonly description = 'Create a new Python script project'; | ||
public readonly tooltip = new MarkdownString('Create a new Python script with basic structure'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similarly here, should we be presenting this as the base "Python project"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so do you think I should just say "project" for the display name then too? Once we get pep 723 then we will look to separate them since the requirements will be done differently?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also do we care these are the same wording:
public readonly description = 'Create a new Python project';
public readonly tooltip = new MarkdownString('Create a new Python project');
f325412
into
microsoft:create-project-branch
No description provided.