jackylee-ch commented on code in PR #15676:
URL: https://github.com/apache/iceberg/pull/15676#discussion_r3033298513
##########
baseline.gradle:
##########
@@ -187,6 +187,21 @@ subprojects {
}
pluginManager.withPlugin('scala') {
+ apply plugin: 'io.github.cosmicsilence.scalafix'
+
+ scalafix {
+ configFile = file("${rootDir}/.baseline/scala/.scalafix.conf")
+ semanticdb {
+ autoConfigure = true
+ }
+ }
+
+ // When running scalafix, downgrade unused import errors to warnings so
that
+ // compilation succeeds and SemanticDB is produced for Scalafix to process.
+ boolean scalafixRun = gradle.startParameter.taskNames.any {
Review Comment:
In fact, introducing Scalafix is a bit heavy, but we [lack this
component](https://github.com/apache/iceberg/pull/14344#issuecomment-3410567850).
Alternatively, do you think we can maintain the current state and let users
handle this code style issue themselves when they receive compilation failures
related to unused imports?
If so, the PR and the issue can be closed now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
