You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2021/08/25 15:31:47 UTC

[GitHub] [zookeeper] ztzg edited a comment on pull request #1735: ZOOKEEPER-4343: Bump commons-io to version 2.11 (avoids CVE-2021-29425)

ztzg edited a comment on pull request #1735:
URL: https://github.com/apache/zookeeper/pull/1735#issuecomment-905613856


   > > would you want me to remove the *.LICENSE.txt?
   >
   > no, please keep it
   
   Okay.
   
   > > . Should we merge this?
   >
   > No if CI is not green, I have restarted the job
   
   Ah, the GitHub CI runs finally went through. (I had restarted the job a couple of times already, but kept hitting random flaky errors.  Isn't the `full-build-java-tests, 11` job kinda redundant wrt. to the Jenkins run, which was already passing?  I obviously wouldn't suggest to ignore CI in general.)
   
   Should I take care of the merge?
   


-- 
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: notifications-unsubscribe@zookeeper.apache.org

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