Skip to content

Conversation

@andy1li
Copy link
Member

@andy1li andy1li commented Oct 7, 2025

Note

Switch Kotlin starter/solutions from Maven to Gradle, update entrypoint and run flow, bump buildpack to kotlin-2.2, and adjust CI/Docker accordingly.

  • Kotlin build and run flow:
    • Replace Maven with Gradle across compiled_starters/kotlin, starter_templates/kotlin, and solutions/kotlin/01-dr6 (.codecrafters/compile.sh, .codecrafters/run.sh, your_program.sh).
    • Add Gradle project files: settings.gradle.kts, app/build.gradle.kts, gradle.properties, gradle/libs.versions.toml.
    • Change output/run to use Gradle distribution binaries under /tmp/codecrafters-build-sqlite-kotlin/distributions.
    • Expand .gitignore with Gradle/Kotlin/IDE artifacts.
  • Project layout and docs:
    • Update entrypoint to app/src/main/kotlin/App.kt and reflect in README.md, explanation.md, and starter config.yml (also switch required_executable to Gradle 9.1.0).
  • Runtime/Build environment:
    • Upgrade buildpack to kotlin-2.2 in codecrafters.yml and add dockerfiles/kotlin-2.2.Dockerfile (Gradle JDK24 base, cache handling).
  • CI:
    • Enhance .github/workflows/test.yml with with: sdkRef: main, permissions, and secrets: inherit.

Written by Cursor Bugbot for commit 25ce411. This will update automatically on new commits. Configure here.

- Replaced Maven with Gradle for build configuration.
- Updated entry point for SQLite implementation to `app/src/main/kotlin/App.kt`.
- Enhanced `.gitignore` to include Gradle-specific files and directories.
- Added `gradle.properties` and `settings.gradle.kts` for Gradle setup.
- Updated `codecrafters.yml` to use Kotlin version 2.2.
- Adjusted shell scripts for Gradle build and execution.
- Removed deprecated `Main.kt` file.
@andy1li andy1li self-assigned this Oct 7, 2025
@linear
Copy link

linear bot commented Oct 7, 2025

- Added `sdkRef` parameter to specify the SDK version.
- Updated job permissions to allow read access to contents and write access for checks and pull requests.
- Configured secrets to inherit from the repository.
@andy1li andy1li merged commit 599dacd into main Oct 7, 2025
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants