[GitHub] [nifi] r65535 commented on pull request #6337: NIFI-7392: Add ValidateJson processor to standard bundle

2022-11-16 Thread GitBox


r65535 commented on PR #6337:
URL: https://github.com/apache/nifi/pull/6337#issuecomment-1317257062

   Thanks for helping us get this over the line @exceptionfactory!


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

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



[GitHub] [nifi] r65535 commented on pull request #6337: NIFI-7392: Add ValidateJson processor to standard bundle

2022-11-16 Thread GitBox


r65535 commented on PR #6337:
URL: https://github.com/apache/nifi/pull/6337#issuecomment-1316944568

   @dan-s1 - thanks for making the changes to this PR! 拾 
   
   @exceptionfactory - do you think we could get this merged in for the 1.19.0 
release?


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

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



[GitHub] [nifi] r65535 commented on pull request #6337: NIFI-7392: Add ValidateJson processor to standard bundle

2022-09-27 Thread GitBox


r65535 commented on PR #6337:
URL: https://github.com/apache/nifi/pull/6337#issuecomment-1259586786

   @exceptionfactory - do you mind taking a look at this PR? I'm keen to get it 
merged into the nifi code base, if possible!


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

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



[GitHub] [nifi] r65535 commented on pull request #6337: NIFI-7392: Add ValidateJson processor to standard bundle

2022-09-01 Thread GitBox


r65535 commented on PR #6337:
URL: https://github.com/apache/nifi/pull/6337#issuecomment-1233933272

   > Thanks for submitting a new pull request for this processor @r65535!
   > 
   > Before reviewing the details, there appears to be a test failure in the 
automated build on Windows.
   
   Fixed - the JSON library supports French and English validation output, and 
my unit tests were validating the English content. My bad!


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

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