ottlinger commented on code in PR #268:
URL: https://github.com/apache/creadur-rat/pull/268#discussion_r1667440124
##########
apache-rat-core/src/main/java/org/apache/rat/Defaults.java:
##########
@@ -143,10 +114,10 @@ public static void init() {
/**
* Builder constructs instances.
* @param log The log to write messages to.
- * @param uris The set of URIs to read.
+ * @param urls The set of URLs to read.
*/
- private Defaults(final Log log, final Set<URI> uris) {
- this.setFactory = Defaults.readConfigFiles(log, uris);
+ private Defaults(final Log log, final Set<URI> urls) {
Review Comment:
I thought the other merge part 1 already fixed this and changed it into uris
instead of urls? Is that a sideeffect of the forced push you did on the
branch8es)?
--
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: dev-unsubscr...@creadur.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
