-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Add App Modernization related documentation for VS Code Java node #8901
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
Conversation
cc @ntrogh |
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.
@nickzhums Left some minor suggestions.
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
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.
@nickzhums There are a few more suggestions pending. Otherwise LGTM.
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Co-authored-by: Nick Trogh <[email protected]>
Thanks @ntrogh ! Committed the changes. Can we merge this PR on tomorrow around 9 - 10 AM PST? |
Please merge this PR after Sep 23rd (We are targeting to GA this product, and why we are updating the docs) - Thanks!