Skip to content

Commit e142783

Browse files
bm1549claude
andcommitted
feat: wire all static check plugins into build.gradle.kts and CI pipeline
- Apply 6 new lint plugins in root build.gradle.kts - Add check_pr_hygiene CI job for shell-based PR checks - All new checks are advisory (warnings only) except checkEmptyInstrumentations Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 7fdad44 commit e142783

2 files changed

Lines changed: 17 additions & 0 deletions

File tree

.gitlab-ci.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -468,6 +468,17 @@ check_build_src:
468468
variables:
469469
GRADLE_TARGET: ":buildSrc:build"
470470

471+
check_pr_hygiene:
472+
extends: .gradle_build
473+
needs: []
474+
stage: tests
475+
script:
476+
- bash scripts/check-ci-debug-flags.sh || true
477+
- bash scripts/check-extraneous-files.sh || true
478+
rules:
479+
- if: $CI_PIPELINE_SOURCE == "merge_request_event"
480+
- if: $CI_COMMIT_BRANCH && $CI_COMMIT_BRANCH != "master"
481+
471482
check_base:
472483
extends: .check_job
473484
variables:

build.gradle.kts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,12 @@ plugins {
99
id("dd-trace-java.tracer-version")
1010
id("dd-trace-java.dump-hanged-test")
1111
id("dd-trace-java.config-inversion-linter")
12+
id("dd-trace-java.empty-instrumentation-linter")
13+
id("dd-trace-java.unnecessary-else-linter")
14+
id("dd-trace-java.naming-convention-linter")
15+
id("dd-trace-java.javadoc-linter")
16+
id("dd-trace-java.copy-paste-detector")
17+
id("dd-trace-java.assertj-preference-linter")
1218
id("dd-trace-java.ci-jobs")
1319

1420
id("com.diffplug.spotless") version "8.2.1"

0 commit comments

Comments
 (0)