docs: update the petclinic readme for mysql support#68
docs: update the petclinic readme for mysql support#68Sarthak160 wants to merge 1 commit intomainfrom
Conversation
Signed-off-by: Sarthak160 <rocksarthak45@gmail.com>
| ``` | ||
| mvn clean install -Dmaven.test.skip=true | ||
| ``` | ||
| : |
There was a problem hiding this comment.
please remove this
| #### Spin up the MySQL database | ||
|
|
||
| ``` | ||
| docker run command - docker run -e MYSQL_USER=mysql -e MYSQL_PASSWORD=123456 -e MYSQL_ROOT_PASSWORD=123456 -e MYSQL_DATABASE=testdb -p 3306:3306 --rm mysql:latest |
There was a problem hiding this comment.
"Docker run command - " should be out of the codeblock.
| spring.datasource.url=jdbc:mysql://localhost:3306/testdb?useSSL=false&allowPublicKeyRetrieval=true | ||
| ``` | ||
|
|
||
| 2) Currently best experience for using MySQL support of keploy is to use default settings which where |
There was a problem hiding this comment.
Please make this point clear or remove it.
|
|
||
| 1) Please note that SSL is currently not supported in the MySQL integration. To use the package without SSL, you can include the following parameters in your database URL like the following example - | ||
| ``` | ||
| spring.datasource.url=jdbc:mysql://localhost:3306/testdb?useSSL=false&allowPublicKeyRetrieval=true |
There was a problem hiding this comment.
Insteading of righting the same code block again, reference the above code block here, or mention this point above itself.
|
|
8 similar comments
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Nice Pr 😄. We currently support Unit Test Generation only for Go projects. |
3 similar comments
|
Nice Pr 😄. We currently support Unit Test Generation only for Go projects. |
|
Nice Pr 😄. We currently support Unit Test Generation only for Go projects. |
|
Nice Pr 😄. We currently support Unit Test Generation only for Go projects. |
2 participants
Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Additional Context (Please include any Screenshots/gifs if relevant)
...
Checklist: