Skip to content

ci: replace Maven Checkstyle with Flint - #2341

Open
zeitlinger wants to merge 10 commits into
prometheus:mainfrom
zeitlinger:agent/test-flint-checkstyle
Open

ci: replace Maven Checkstyle with Flint#2341
zeitlinger wants to merge 10 commits into
prometheus:mainfrom
zeitlinger:agent/test-flint-checkstyle

Conversation

@zeitlinger

@zeitlinger zeitlinger commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

  • use the released Flint 0.22.10 Checkstyle implementation
  • add Checkstyle 13.8.0 as the direct checkstyle mise entry
  • update CI to mise 2026.7.15 so that direct registry entry is available in every workflow
  • allow the verified @yarnpkg/libzip 3.2.2 release for Renovate because it was published without provenance
  • remove the Maven Checkstyle plugin and its obsolete skip settings so Checkstyle runs only once, through Flint
  • preserve the repository's previous main-source scope with narrow suppressions for test and archived Java sources
  • suppress VariableDeclarationUsageDistance for examples, where setup declarations intentionally precede simulated work

Flint Checkstyle support is released in v0.22.10, and this PR uses the standard aqua:grafana/flint backend for Flint itself.

Testing

  • mise exec -- flint run --full checkstyle
  • mise run lint:fix
  • mise run build

@zeitlinger

Copy link
Copy Markdown
Member Author

This now removes the Maven Checkstyle plugin to avoid running Checkstyle twice. The PR uses the Aqua package from aquaproj/aqua-registry#57625; CI installation depends on that registry PR merging.

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@zeitlinger
zeitlinger force-pushed the agent/test-flint-checkstyle branch from 5a0cbf1 to c949ba4 Compare July 28, 2026 10:26
@zeitlinger zeitlinger changed the title ci: validate unreleased Flint Checkstyle support ci: replace Maven Checkstyle with Flint Jul 28, 2026
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@zeitlinger
zeitlinger marked this pull request as ready for review July 28, 2026 11:44
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@zeitlinger

Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

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