Azure-Samples / java-microservices-aca-lab

Spring Petclinic Microservices with AI on Azure Container Apps
https://aka.ms/aca-lab
MIT License
7 stars 31 forks source link

Givermei spring update version to 3.2.7 #44

Open vermegi opened 2 months ago

vermegi commented 2 months ago

Purpose

fixes #33 and kinda #31

Does this introduce a breaking change?

[ ] Yes
[ x ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ x ] Other... Please describe: New copy of the base repo at https://github.com/spring-petclinic/spring-petclinic-microservices/tree/main

How to Test

Run-through the full lab with this new version

What to Check

Verify that the following are valid

Other Information

I did not do a test run yet, this is just a clean copy paste from https://github.com/spring-petclinic/spring-petclinic-microservices/tree/main

vermegi commented 2 months ago

Btw, this PR will also undo the code changes done for the messaging lab ... @sonwan2020 Could we redo those changes on a separate branch?

vermegi commented 2 months ago

@sonwan2020 can you review? Just to know as well if it's ok to override the full existing code. All edits in code will get lost with this PR.

sonwan2020 commented 2 months ago

@vermegi I will merge and test this PR lately before October?

sonwan2020 commented 1 month ago

some upcoming PRs already upgrade this to 3.3.4 we may delay this