exceptionfactory commented on pull request #5355:
URL: https://github.com/apache/nifi/pull/5355#issuecomment-913809834


   > Could be because I was running on Windows 11 (preview), that may have 
confused the Assume. As I've only that version of Windows to build from 
(normally I build from ubuntu), I don't have any other way to test. I imagine 
the GitHub CI run could be leaving the artefact and noone notices because then 
things get thrown away? It was, however, a brand new clone of the repo and 
first run through of the tests (after initially building with -DskipTests)
   
   Thanks for the reply, @ChrisSamo632, that's interesting regarding Windows 
11. Using `File.separator()` seems reasonable either way, so the changes look 
good.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to