TobKed commented on pull request #12452: URL: https://github.com/apache/beam/pull/12452#issuecomment-672344986
> Thanks, @TobKed , I don't have additional comments. Once all reviewers, whose feedback you expect, give their LGTM, please squash fixup commits, and leave only the commits that you want to be added to the commit history. I found I missed to use `TemporaryDirectory()` in `sdks/python/apache_beam/typehints/typecheck_test_py3.py`, I fixed it. I also added input to manual trigger so dataflow tests have to be explicitly enabled (made them disabled by default). Thanks for reviews @tvalentyn and all people involved in this PR. I will create PR to Apache Infra with request for setting up proper secrets. ---------------------------------------------------------------- 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]
