[GitHub] [commons-imaging] Marcono1234 commented on pull request #307: Use static imports for JUnit Assertions methods

["Marcono1234 (via GitHub)" <gi...@apache.org>]

Marcono1234 commented on PR #307:
URL: https://github.com/apache/commons-imaging/pull/307#issuecomment-1648488839

   > Don't change the order of import statements please
   
   I think all the changes in this pull request follow the existing import ordering, though please point me to a specific location where that is not true in case I overlooked something.
   
   Since this pull request changes the test code to consistently only use `static` imports for JUnit's `Assertions` methods, it has to replace the regular `import` statements with `import static`, which therefore also has to change the "order" (actually not really, because the imports themselves change) to keep following the existing import ordering, that is, `static` imports first.


-- 
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-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org