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

More detailed language pack extension description #677

Merged
merged 3 commits into from
Jun 22, 2021

Conversation

nickzhums
Copy link
Member

By looking at the description of other language extensions, our java extension pack description is quite simple, so adding a few more details to provide more clarity for developers

@nickzhums nickzhums marked this pull request as draft June 22, 2021 02:31
@nickzhums nickzhums marked this pull request as ready for review June 22, 2021 02:32
package.json Outdated
@@ -55,6 +55,7 @@
"onWebviewPanel:java.welcome",
"onCommand:java.classpathConfiguration",
"onWebviewPanel:java.classpathConfiguration",
"onWebviewPanel:java.markdownPreview",
Copy link
Member

Choose a reason for hiding this comment

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

what's this line for? @jdneo please take a look

Copy link
Member Author

Choose a reason for hiding this comment

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

i don't think i edited this line but somehow it's showing up here, i can remove it if helps

Copy link
Member

Choose a reason for hiding this comment

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

This line has been removed in a recent commit. Feel safe to remove it

Copy link
Member Author

Choose a reason for hiding this comment

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

just removed

package.json Outdated
@@ -1,7 +1,7 @@
{
"name": "vscode-java-pack",
"displayName": "Java Extension Pack",
"description": "Popular extensions for Java development and more.",
"description": "Popular extensions for Java development that provides Java intellisense, debugging, testing, Maven support, project management and more",
Copy link
Contributor

Choose a reason for hiding this comment

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

intellisense -> IntelliSense. It's a general term. See https://code.visualstudio.com/docs/editor/intellisense

Maven support -> Maven/Gradle support

Copy link
Member Author

Choose a reason for hiding this comment

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

great

@nickzhums
Copy link
Member Author

anyone can help to merge this PR? I don't have write access : ( thanks~

@jdneo jdneo merged commit 0b56bac into microsoft:main Jun 22, 2021
@jdneo
Copy link
Member

jdneo commented Jun 22, 2021

Thank you @nickzhums

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants