8000 Simplify PR builds by cmgrote · Pull Request #2120 · atlanhq/atlan-java · GitHub
[go: up one dir, main page]

Skip to content

Simplify PR builds#2120

Merged
cmgrote merged 4 commits intomainfrom
dependencies
Dec 4, 2025
Merged

Simplify PR builds#2120
cmgrote merged 4 commits intomainfrom
dependencies

Conversation

@cmgrote
Copy link
Collaborator
@cmgrote cmgrote commented Dec 4, 2025

Note

Streamlines CI by switching to the new Gradle setup action, consolidating formatting into the PR build, refining unit test execution, and enforcing Spotless ordering in build scripts.

  • CI workflows:
    • Replace gradle/gradle-build-action with gradle/actions/setup-gradle@v5 and enable add-job-summary: on-failure across PR and Test workflows.
    • Consolidate formatting into pr-build (remove separate pr-format job) and keep spotlessCheck step.
    • pr-test: enable Gradle caching options, and run ./gradlew testClasses test for unit tests.
  • Build script (buildSrc/src/main/kotlin/com.atlan.kotlin-custom-package.gradle.kts):
    • Import SpotlessTask and ensure code formatting runs after package generation: genCustomPkg finalizedBy("spotlessApply"), SpotlessTask mustRunAfter("genCustomPkg"), and make spotlessCheck depend on classes.

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

Signed-off-by: Chris (He/Him) <cgrote@gmail.com>
Signed-off-by: Chris (He/Him) <cgrote@gmail.com>
@cmgrote cmgrote enabled auto-merge December 4, 2025 14:29
Signed-off-by: Chris (He/Him) <cgrote@gmail.com>
Signed-off-by: Chris (He/Him) <cgrote@gmail.com>
@cmgrote cmgrote merged commit d76d398 into main Dec 4, 2025
6 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.

1 participant

0