Skip to content

Commit f713848

Browse files
committed
remove unnecessary semicolons from groovy files
1 parent 0cb89f2 commit f713848

3 files changed

Lines changed: 10 additions & 10 deletions

File tree

src/main/groovy/org/codehaus/mojo/spotbugs/BaseViolationCheckMojo.groovy

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -197,19 +197,19 @@ abstract class BaseViolationCheckMojo extends AbstractMojo {
197197
}
198198

199199
private boolean doSourceFilesExist() {
200-
boolean foundClassFiles = false;
200+
boolean foundClassFiles = false
201201
List<String> classFilesList = []
202202
if (this.classFilesDirectory.isDirectory()) {
203203
foundClassFiles = walkFiles(classFilesList, classFilesDirectory, foundClassFiles)
204204
}
205205

206-
boolean foundTestFiles = false;
206+
boolean foundTestFiles = false
207207
List<String> testFilesList = []
208208
if (this.includeTests && this.testClassFilesDirectory.isDirectory()) {
209209
foundTestFiles = walkFiles(testFilesList, testClassFilesDirectory, foundTestFiles)
210210
}
211211

212-
return foundClassFiles || foundTestFiles;
212+
return foundClassFiles || foundTestFiles
213213
}
214214

215215
private boolean walkFiles(List filesList, File filesDirectory, boolean foundFiles) {

src/main/groovy/org/codehaus/mojo/spotbugs/ResourceHelper.groovy

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,14 +93,14 @@ final class ResourceHelper {
9393
if (new File(name).exists()) {
9494
// Avoid copying the file onto itself
9595
if (Path.of(name).toAbsolutePath().normalize().equals(outputResourcePath.toAbsolutePath().normalize())) {
96-
return outputResourcePath;
96+
return outputResourcePath
9797
}
9898

9999
createParentDirectories(outputResourcePath)
100100

101101
// Copy existing file (not a URL)
102102
return Files.copy(Path.of(name), outputResourcePath, StandardCopyOption.REPLACE_EXISTING,
103-
StandardCopyOption.COPY_ATTRIBUTES);
103+
StandardCopyOption.COPY_ATTRIBUTES)
104104
}
105105

106106
// Copying resource from classpath to a file

src/main/groovy/org/codehaus/mojo/spotbugs/SpotBugsMojo.groovy

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -563,25 +563,25 @@ class SpotBugsMojo extends AbstractMavenReport implements SpotBugsPluginsTrait {
563563
log.debug("canGenerate is ${canGenerate}")
564564
}
565565

566-
boolean isSiteLifecycle = false;
566+
boolean isSiteLifecycle = false
567567
if (session != null && session.getRequest() != null) {
568-
List<String> goals = session.getRequest().getGoals();
568+
List<String> goals = session.getRequest().getGoals()
569569
if (goals != null && goals.any { String goal -> goal == "site" || goal.startsWith("site:") }) {
570-
isSiteLifecycle = true;
570+
isSiteLifecycle = true
571571
}
572572
}
573573

574574
if (canGenerate) {
575575
if (!isSiteLifecycle) {
576576
// Only generate xdoc report, skip site pages
577577
generateXDoc(getLocale())
578-
return false;
578+
return false
579579
}
580580
} else {
581581
log.info('No files found to run spotbugs, check compile phase has been run')
582582
}
583583

584-
return canGenerate;
584+
return canGenerate
585585
}
586586

587587
/**

0 commit comments

Comments
 (0)