-
-
Notifications
You must be signed in to change notification settings - Fork 270
docs: Update the java Quickstart and images #684
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
Signed-off-by: kenk999 <mohammadhifzan24@gmail.com>
Signed-off-by: Pratik Mahalle <mahallepratik683@gmail.com>
Signed-off-by: Pratik Mahalle <mahallepratik683@gmail.com>
Signed-off-by: Pratik Mahalle <mahallepratik683@gmail.com>
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.
Pull Request Overview
Updates the Java quickstart documentation and images to improve clarity and accuracy of the documentation.
- Updates image references to use more descriptive filenames
- Fixes documentation headers and text formatting
- Streamlines command instructions by combining git clone and cd operations
Reviewed Changes
Copilot reviewed 7 out of 22 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| versioned_docs/version-3.0.0/server/installation_tabs.md | Updates Docker installation header for clarity on WSL |
| versioned_docs/version-3.0.0/quickstart/samples-java.md | Updates image references to use more descriptive filenames |
| versioned_docs/version-3.0.0/quickstart/java-spring-postgres.md | Updates image references for PostgreSQL quickstart |
| versioned_docs/version-3.0.0/quickstart/java-spring-boot-xml.md | Replaces GitHub asset images with local documentation images |
| versioned_docs/version-3.0.0/quickstart/java-spring-boot-openhospital.md | Adds cd commands and updates image references |
| versioned_docs/version-3.0.0/keploy-explained/testing-guide.md | Fixes header formatting |
| versioned_docs/version-2.0.0/quickstart/java-spring-boot-openhospital.md | Combines git clone and cd commands |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| git clone https://github.com/keploy/openhospital-core | ||
| cd openhospital-core/ |
Copilot
AI
Sep 24, 2025
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 git clone and cd commands should be combined into a single line for consistency with the pattern used in version 2.0.0, as git clone https://github.com/keploy/openhospital-core && cd openhospital-core/.
| git clone https://github.com/keploy/openhospital-core | |
| cd openhospital-core/ | |
| git clone https://github.com/keploy/openhospital-core && cd openhospital-core/ |
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.
Fix the Copilot review @pratik-mahalle
| git clone https://github.com/keploy/openhospital-api | ||
| cd openhospital-api/ |
Copilot
AI
Sep 24, 2025
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 git clone and cd commands should be combined into a single line for consistency, as git clone https://github.com/keploy/openhospital-api && cd openhospital-api/.
| git clone https://github.com/keploy/openhospital-api | |
| cd openhospital-api/ | |
| git clone https://github.com/keploy/openhospital-api && cd openhospital-api/ |
| git clone https://github.com/keploy/openhospital-ui | ||
| cd openhospital-ui/ |
Copilot
AI
Sep 24, 2025
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 git clone and cd commands should be combined into a single line for consistency, as git clone https://github.com/keploy/openhospital-ui && cd openhospital-ui/.
| git clone https://github.com/keploy/openhospital-ui | |
| cd openhospital-ui/ | |
| git clone https://github.com/keploy/openhospital-ui && cd openhospital-ui/ |
Signed-off-by: Pratik Mahalle <mahallepratik683@gmail.com>
Achanandhi-M
left a comment
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.
LGTM
| In this blog, we will learn how test bench of keploy works and how you can contribute to keploy's code coverage. | ||
|
|
||
| ## Keploy test Keploy? | ||
| ## Keploy Test? |
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.
In this we don't want to change this line @pratik-mahalle
| git clone https://github.com/keploy/openhospital-core | ||
| cd openhospital-core/ |
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.
Fix the Copilot review @pratik-mahalle
| ## Install Keploy with Docker on Windows | ||
| ## Install Keploy with Docker on WSL | ||
|
|
||
| 1. **Make sure Docker is installed** : You’ll need **Docker Desktop** running on Windows. |
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.
Can you please remove the Docker Desktop running on windows ? Include only Docker installed. Also update in other places where windows is used changed to wsl in the docker installation for windows
What has changed?
Please include a summary of the change.
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please run npm run build and npm run serve to check if the changes are working as expected. Please include screenshots of the output of both the commands. Add screenshots/gif of the changes if possible.
Checklist: