Pankraz76 commented on PR #2277: URL: https://github.com/apache/maven/pull/2277#issuecomment-2869215629
> Regarding point 2. (and the related point 4.): I consider the IAE more concise than an NPE in case a method argument is passed with null (although not allowed). Compare also with https://discuss.kotlinlang.org/t/why-does-requirenotnull-throw-an-illegalargumentexception/7617/4 Java and Lombok prefer NPE as well. https://docs.oracle.com/javase/8/docs/api/java/util/Objects.html#requireNonNull-T- https://projectlombok.org/features/NonNull both valid points. Ideally its just one config-switch to adjust: <img width="749" alt="image" src="https://github.com/user-attachments/assets/ebff5182-0f9e-4346-adf0-9dce53412e82"; /> -- 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: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
