Skip to content

Commit

Permalink
fix(common-jvm): improve the check for presence of Maven Wrapper nece…
Browse files Browse the repository at this point in the history
…ssary files

The check was too strict and would not consider Maven Wrapper to be present if some optional files were missing.

Closes #230
  • Loading branch information
tinesoft committed Jul 21, 2024
1 parent 2d2017b commit c0a2ed3
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions packages/common-jvm/src/lib/utils/maven-utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -556,11 +556,6 @@ function hasMavenWrapperWithPredicate(predicate: (file: string) => boolean) {
return (
['mvnw', '.mvn/wrapper/maven-wrapper.properties'].every((file) =>
predicate(file)
) &&
['mvnw.bat', 'mvnw.cmd'].some((file) => predicate(file)) &&
[
'.mvn/wrapper/MavenWrapperDownloader.java',
'.mvn/wrapper/maven-wrapper.jar',
].some((file) => predicate(file))
) && ['mvnw.bat', 'mvnw.cmd'].some((file) => predicate(file))
);
}

0 comments on commit c0a2ed3

Please sign in to comment.