-
Notifications
You must be signed in to change notification settings - Fork 143
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
Support for VS Code #261
Support for VS Code #261
Conversation
Please fix all file names to eliminate spaces. Use dashes, hyphens or no spaces at all. Using spaces will cause havoc in non-Windows systems. |
Please fix all directory/path/file names to eliminate spaces. Use dashes, hyphens or no spaces at all. Using spaces will cause havoc in non-Windows systems. |
VS Code/readme.md
Outdated
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.
Please fix all directory/path/file names to eliminate spaces. Use dashes, hyphens or no spaces at all. Using spaces will cause havoc in non-Windows systems.
Changed the file names and eliminated the spaces in the lastest commit. |
Please examine my prior comments carefully. It wasn’t just file names that was a problem. Unless the comments are fully addressed, I won’t look at this PR. |
VS_Code/readme.md
Outdated
![import3](<images/importing_cargotracker_3.png>) | ||
|
||
### Building and Running on Payara 5 | ||
1. After the project loads,Go under the maven tab on the left bottom side,the eclipse cargo tracker name will be shown.Right-click on it and proceed with run maven commands clean,package,install and compile(which will take a while the very first time as Maven downloads dependencies) |
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.
The exact Maven goals that need to be executed are clean and package. Please fix this.
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.
This should show Payara 5, not Payara 6.
Description
This PR adds a documentation file of the cargo tracker in Visual Studio Code.
Added support for VS Code IDE.
Covers this issue as well -:
Support VS Code, IntelliJ #201 Support VS Code, IntelliJ.