search

kordamp / pomchecker

🦉 Maven POM syntax checker
Apache License 2.0
60 stars 8 forks source link

chore: Use the pom.xml in the current directory as a default value for --file #14

Closed helpermethod closed 1 year ago

helpermethod commented 1 year ago

Relates to #12.

One problem with the default value is obviously that it throws at you an ugly stacktrace if there is no pom.xml file in the current directory.

I guess it makes sense to check for the existence and present a nicer error message?

aalmiray commented 1 year ago

Agreed. Adding a check before executing the real command would be ideal.

aalmiray commented 1 year ago

Build failed because PRs don't have access to repo vars 😠

helpermethod commented 1 year ago

The file parameter is now validated and shows an error if the pom.xml (which is either provided by --file or defaulted to pom.xml) does not exist.