-
Notifications
You must be signed in to change notification settings - Fork 0
ci: update generated copier template from example #82
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
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
This automated PR attempts to update the generated copier template by migrating to Spring Boot 4.0.0 and reorganizing package imports. However, this PR contains critical issues that will prevent the project from building or running.
Key Issues:
- Attempts to use Spring Boot 4.0.0, which does not exist (latest is 3.x series as of January 2025)
- Introduces non-existent packages and artifacts (
spring-boot-resttestclient,spring-boot-starter-data-jpa-test, etc.) - Changes imports to invalid package locations that don't exist in any Spring Boot release
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 14 comments.
Show a summary per file
| File | Description |
|---|---|
{{app_name}}/pom.xml.jinja |
Updates dependency versions to non-existent versions (Spring Boot 4.0.0, springdoc 3.0.0, pre-liquibase 2.0.0); adds Lombok annotation processor configuration; removes h2 and snakeyaml version properties |
{{app_name}}/rest-adapter/pom.xml.jinja |
Adds non-existent spring-boot-resttestclient dependency |
{{app_name}}/rest-adapter/src/test/java/{{package_name}}/rest/{{domain_capitalized}}ResourceTest.java.jinja |
Changes TestRestTemplate imports to non-existent package; adds non-existent @AutoConfigureTestRestTemplate annotation; removes valid DataSourceAutoConfiguration exclusion |
{{app_name}}/jpa-adapter/pom.xml.jinja |
Changes liquibase dependency structure; adds non-existent spring-boot-starter-data-jpa-test dependency |
{{app_name}}/jpa-adapter/src/main/java/{{package_name}}/repository/config/JpaAdapterConfig.java.jinja |
Changes EntityScan import to non-existent package |
{{app_name}}/jpa-adapter/src/test/java/{{package_name}}/repository/{{domain_capitalized}}JpaTest.java.jinja |
Changes DataJpaTest import to non-existent package |
{{app_name}}/jpa-adapter/src/test/java/{{package_name}}/repository/{{domain_capitalized}}JpaAdapterApplication.java.jinja |
Removes PreLiquibaseAutoConfiguration import and usage |
{{app_name}}/jpa-adapter/src/main/resources/preliquibase/h2.sql.jinja |
Adds new H2 pre-liquibase SQL script with schema creation statements |
{{app_name}}/acceptance-test/pom.xml.jinja |
Adds non-existent spring-boot-resttestclient dependency |
{{app_name}}/acceptance-test/src/test/java/{{package_name}}/cucumber/{{domain_capitalized}}StepDef.java.jinja |
Changes TestRestTemplate import to non-existent package |
{{app_name}}/acceptance-test/src/test/java/{{package_name}}/cucumber/SpringCucumberTestConfig.java.jinja |
Adds non-existent @AutoConfigureTestRestTemplate annotation and import |
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
Automated PR created by workflow.