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

Support for VS Code #261

Merged
merged 7 commits into from
Jul 2, 2023
Merged

Conversation

ojuschugh1
Copy link
Contributor

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.

@m-reza-rahman
Copy link
Contributor

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.

@m-reza-rahman
Copy link
Contributor

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.

Copy link
Contributor

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.

@ojuschugh1
Copy link
Contributor Author

Changed the file names and eliminated the spaces in the lastest commit.

@m-reza-rahman
Copy link
Contributor

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.

![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)
Copy link
Contributor

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.

Copy link
Contributor

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.

@m-reza-rahman m-reza-rahman merged commit 6b107f6 into eclipse-ee4j:master Jul 2, 2023
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.

2 participants