Abacn commented on PR #24212: URL: https://github.com/apache/beam/pull/24212#issuecomment-1317724105
Thanks @creste. Left some comments below the issue page. For the tests, to fix Lint and Formatter error, one could do ``` # Run from root beam repo dir pip install yapf==0.29.0 git diff master --name-only | grep "\.py$" | xargs yapf --in-place ``` or ``` # Run from sdks/python tox -e py3-yapf ``` For RAT error, adding apache license at the top of new files (see other source files in the project) -- 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]
