Improve duplicate clinit removal - #10383
Draft
niloc132 wants to merge 14 commits into
Draft
Annotations
17 warnings
|
Create pull request comments/annotations for Checkstyle and ErrorProne from the java 21 build, even on failure:
dev/core/test/com/google/gwt/dev/js/OptimizerTestBase.java#L26
[checkstyle] reported by reviewdog 🐶
'junit.framework.TestCase' should be separated from previous imports.
Raw Output:
/home/runner/work/gwt/gwt/gwt/dev/core/test/com/google/gwt/dev/js/OptimizerTestBase.java:26:1: warning: 'junit.framework.TestCase' should be separated from previous imports. (com.puppycrawl.tools.checkstyle.checks.imports.ImportOrderCheck)
|
|
Create pull request comments/annotations for Checkstyle and ErrorProne from the java 21 build, even on failure:
dev/core/test/com/google/gwt/dev/js/OptimizerTestBase.java#L25
[checkstyle] reported by reviewdog 🐶
Extra separation in import group before 'com.google.gwt.thirdparty.guava.common.base.Joiner'
Raw Output:
/home/runner/work/gwt/gwt/gwt/dev/core/test/com/google/gwt/dev/js/OptimizerTestBase.java:25:1: warning: Extra separation in import group before 'com.google.gwt.thirdparty.guava.common.base.Joiner' (com.puppycrawl.tools.checkstyle.checks.imports.ImportOrderCheck)
|
|
Create pull request comments/annotations for Checkstyle and ErrorProne from the java 21 build, even on failure:
dev/core/test/com/google/gwt/dev/js/JsStaticEvalTest.java#L155
[checkstyle] reported by reviewdog 🐶
Line matches the illegal pattern 'more than one blank line'.
Raw Output:
/home/runner/work/gwt/gwt/gwt/dev/core/test/com/google/gwt/dev/js/JsStaticEvalTest.java:155:0: info: Line matches the illegal pattern 'more than one blank line'. (com.puppycrawl.tools.checkstyle.checks.regexp.RegexpCheck)
|
|
Create pull request comments/annotations for Checkstyle and ErrorProne from the java 21 build, even on failure:
dev/core/test/com/google/gwt/dev/js/JsDuplicateFunctionRemoverTest.java#L77
[checkstyle] reported by reviewdog 🐶
Line matches the illegal pattern 'more than one blank line'.
Raw Output:
/home/runner/work/gwt/gwt/gwt/dev/core/test/com/google/gwt/dev/js/JsDuplicateFunctionRemoverTest.java:77:0: info: Line matches the illegal pattern 'more than one blank line'. (com.puppycrawl.tools.checkstyle.checks.regexp.RegexpCheck)
|
|
Create pull request comments/annotations for Checkstyle and ErrorProne from the java 21 build, even on failure:
dev/core/test/com/google/gwt/dev/js/JsDuplicateFunctionRemoverTest.java#L33
[checkstyle] reported by reviewdog 🐶
Line matches the illegal pattern 'more than one blank line'.
Raw Output:
/home/runner/work/gwt/gwt/gwt/dev/core/test/com/google/gwt/dev/js/JsDuplicateFunctionRemoverTest.java:33:0: info: Line matches the illegal pattern 'more than one blank line'. (com.puppycrawl.tools.checkstyle.checks.regexp.RegexpCheck)
|
|
Create pull request comments/annotations for Checkstyle and ErrorProne from the java 21 build, even on failure:
dev/core/src/com/google/gwt/dev/js/DuplicateClinitRemover.java#L286
[checkstyle] reported by reviewdog 🐶
Line is longer than 100 characters (found 101).
Raw Output:
/home/runner/work/gwt/gwt/gwt/dev/core/src/com/google/gwt/dev/js/DuplicateClinitRemover.java:286:0: warning: Line is longer than 100 characters (found 101). (com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck)
|
|
Create pull request comments/annotations for Checkstyle and ErrorProne from the java 21 build, even on failure:
dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java#L3818
[checkstyle] reported by reviewdog 🐶
Line is longer than 100 characters (found 103).
Raw Output:
/home/runner/work/gwt/gwt/gwt/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java:3818:0: warning: Line is longer than 100 characters (found 103). (com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck)
|
|
Build, style/api checks, produce docs
[removal] Utility in com.google.gwt.util.tools has been deprecated and marked for removal
|
|
Build, style/api checks, produce docs
[strictfp] as of release 17, all floating-point expressions are evaluated strictly and 'strictfp' is not required
|
|
Build, style/api checks, produce docs
[removal] Double(double) in Double has been deprecated and marked for removal
|
|
Build, style/api checks, produce docs
[removal] Double(double) in Double has been deprecated and marked for removal
|
|
Build, style/api checks, produce docs
[removal] Double(double) in Double has been deprecated and marked for removal
|
|
Build, style/api checks, produce docs
[removal] Double(double) in Double has been deprecated and marked for removal
|
|
Build, style/api checks, produce docs
[removal] Double(double) in Double has been deprecated and marked for removal
|
|
Build, style/api checks, produce docs
[LongDoubleConversion] Conversion from long to double may lose precision; use an explicit cast to double if this was intentional
|
|
Build, style/api checks, produce docs
[CatchAndPrintStackTrace] Logging or rethrowing exceptions should usually be preferred to catching and calling printStackTrace
|
|
Build, style/api checks, produce docs
[VoidUsed] Using a Void-typed variable is potentially confusing, and can be replaced with a literal `null`.
|
background
wait
wait-all
cancel
parallel
Loading