Hannah-Jiang commented on a change in pull request #11428:
URL: https://github.com/apache/beam/pull/11428#discussion_r411765855
##########
File path: sdks/java/container/license_scripts/license_script.sh
##########
@@ -25,15 +25,23 @@ pip install PyYAML==5.3
pip install tenacity==5.0.4
Review comment:
I reviewed to use tox here, but it is a little complicated. We want to
keep it simple within a script, so it's easy to use and maintain. Current
scripts run at local machine, (not within a docker), so docker will still have
clear environment.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
