muralibasani commented on code in PR #21963:
URL: https://github.com/apache/kafka/pull/21963#discussion_r3212896704


##########
release/github.py:
##########
@@ -0,0 +1,128 @@
+#

Review Comment:
   @mimaison  with this change, there are multiple dependencies which should be 
added to requirements, which also include cryptography, pynacl, cffi and these 
have native C/rust compile steps (which might fail on release mgr machines just 
in case) if rel manager machine doesn't have wheels installed.   (Usually PyPI 
ships prebuilt wheels)
   
   If these are still reasonable to continue, I shall update pr using PyGithub 
instead.



-- 
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