Claudenw commented on code in PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#discussion_r1587463735
##
apache-rat-core/src/main/java/org/apache/rat/config/parameters/ComponentType.java:
##
@@ -28,6 +28,6 @@ public enum ComponentType {
MATCHER,
/** A Param
ottlinger commented on code in PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#discussion_r1579991867
##
apache-rat-core/spotbugs_ignore.xml:
##
@@ -0,0 +1,27 @@
+
+
+
+
+
ottlinger commented on code in PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#discussion_r1579940463
##
apache-rat-core/spotbugs_ignore.xml:
##
@@ -0,0 +1,27 @@
+
+
+
+
+
ottlinger commented on code in PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#discussion_r1579989686
##
apache-rat-tasks/src/main/java/org/apache/rat/anttasks/Report.java:
##
@@ -318,7 +319,6 @@ public void log(Level level, String msg) {
write(Proj
ottlinger commented on code in PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#discussion_r1579942537
##
apache-rat-plugin/spotbugs_ignore.xml:
##
@@ -0,0 +1,27 @@
+
+
+
+
+
ottlinger commented on code in PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#discussion_r1579940463
##
apache-rat-core/spotbugs_ignore.xml:
##
@@ -0,0 +1,27 @@
+
+
+
+
+
Claudenw commented on code in PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#discussion_r1579490672
##
apache-rat-core/spotbugs_ignore.xml:
##
@@ -0,0 +1,27 @@
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
ottlinger commented on PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#issuecomment-2068186230
Most of the errors relate to the fact that many file operations rely on the
default charset. Unfortunately newer version of the JDK provide constructors to
specify an encoding, but as
ottlinger commented on PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#issuecomment-2067991533
@Claudenw the thing is that it either breaks the build or not. One can
configure exclusions in an XML file in order to tolerate certain bugs
explicitly . I'll have a look, but the
Claudenw commented on PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#issuecomment-2067961771
My thought was to turn on the spotbugs and fix any critical errors at that
time (the ones with the red circles).
We should then open fixes to remove the yellow ones and/or determine
ottlinger commented on PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#issuecomment-2059927715
In order to be able to bring changes intro master I'd prefer to configure
limits per module with the current threshold, thus we could make sure that no
new stuff is being added and ca
ottlinger commented on PR #238:
URL: https://github.com/apache/creadur-rat/pull/238#issuecomment-2059919599
@Claudenw in case of any errors the build fails. I'm not sure if it's that
helpful or if we remove this setting on the branch .
--
This is an automated message from the Apache
ottlinger opened a new pull request, #238:
URL: https://github.com/apache/creadur-rat/pull/238
I verified locally (via ./.buildtools/generateStagingSiteInWebpageRepo) that
a report is generated, but the build is not yet configured to fail.
--
This is an automated message from the Apache G
13 matches
Mail list logo