Skip to content

Conversation

@nakamura-to
Copy link
Member

Summary

  • Add dedicated test modules for H2 and Testcontainers to improve plugin testing
  • Remove duplicate Gradle wrapper files from the codegen module
  • Update CI workflows to use the official Gradle setup action

Test plan

  • CI builds pass with the new test modules
  • Code generation works correctly for both H2 and Testcontainers configurations
  • Gradle wrapper executes properly in CI with the new setup action

🤖 Generated with Claude Code

nakamura-to and others added 3 commits June 29, 2025 09:10
- Restructure project as Gradle composite build with root + included codegen module
- Add codegen-h2-test module for testing with in-memory H2 database
- Add codegen-tc-test module for testing with PostgreSQL via Testcontainers
- Update CI workflow to use composite build structure
- Migrate all modules to use Gradle version catalog
- Update documentation to reflect new multi-module architecture
- Remove obsolete template/entity.ftl file

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
The codegen module had its own Gradle wrapper files which are redundant since the root project already provides them. This consolidates the build to use a single set of wrapper files at the project root.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Replace manual gradlew permission and caching with the official
gradle/actions/setup-gradle action for better maintainability.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@github-actions github-actions bot added the test label Jun 29, 2025
@nakamura-to nakamura-to merged commit d814b7c into master Jun 29, 2025
2 checks passed
@nakamura-to nakamura-to deleted the test/add-h2-test-project branch June 29, 2025 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants