Skip to content

Commit

Permalink
Improve quark parsing message (#672)
Browse files Browse the repository at this point in the history
Fixes #671.

As suggested in the discussion, change the message to indicate that
quark files are not parsed.

Also change the unit from "Mb" to "MB", since "Mb" indicates Megabit.
Add a blank between numbers and unit for readability and to be
consistent with output in other parts of the code.
  • Loading branch information
Bananeweizen authored Dec 2, 2023
1 parent 159d60f commit d99ab8e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/main/java/org/openrewrite/maven/ResourceParser.java
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,8 @@ public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) {
if (!attrs.isOther() && !attrs.isSymbolicLink() &&
!alreadyParsed.contains(file) && !isExcluded(file)) {
if (isOverSizeThreshold(attrs.size())) {
logger.info("Parsing as quark " + file + " as its size " + attrs.size() / (1024L * 1024L) +
"Mb exceeds size threshold " + sizeThresholdMb + "Mb");
logger.info("Not parsing quark " + file + " as its size " + attrs.size() / (1024L * 1024L) +
" MB exceeds size threshold " + sizeThresholdMb + " MB");
quarkPaths.add(file);
} else if (isParsedAsPlainText(file)) {
plainTextPaths.add(file);
Expand Down

0 comments on commit d99ab8e

Please sign in to comment.