Skip to content

java, scala, kotlin: warnings-as-errors through the repo flag and per-target attribute (closes #6180) - #6434

Merged
TrapsterDK merged 1 commit into
mainfrom
jvm-warnings-6180
Nov 4, 2026
Merged

TrapsterDK merged 1 commit into
mainfrom
jvm-warnings-6180

Conversation

@TrapsterDK

Copy link
Copy Markdown
Collaborator

Closes #6180

Java and Scala add -Werror / -Xfatal-warnings through dx_warnings_as_errors; -Xlint:all stays. Kotlin has one options rule with a werror bool; the shared default sets it from the helper, and <name>_kotlinc_opts is declared when kotlinc_opts or treat_warnings_as_errors is set. Added a Kotlin analysistest with the flag off (warn stays report) and on.

Notes

  • The per-language READMEs named in the issue do not exist; only the java-app fixture README sentence was updated.
  • Tests run: java/scala/kotlin rules, custom_opts fixture, libs/starlark. A pre-existing buildifier warning in libs/starlark/wrapper.bzl (dx_macro_upstream_visibility) is not from this change.
  • No nested-Bazel or consumer tests were run.

Assisted by Claude.

🤖 Generated with Claude Code

https://claude.ai/code/session_011u9TXHDZrfmzFQ2WT8QAKK

…-target attribute (part of #6021)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011u9TXHDZrfmzFQ2WT8QAKK
@TrapsterDK
TrapsterDK merged commit 1882fc0 into main Nov 4, 2026
1 check failed
@TrapsterDK
TrapsterDK deleted the jvm-warnings-6180 branch November 4, 2026 19:51
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.

java, scala, kotlin: warnings-as-errors through the repo flag and per-target attribute (part of #6021)

1 participant