Re: [PR] Add a subsuite of unit tests for ML package and remove them from 'coverage' GHA workflow. [beam]

["tvalentyn (via GitHub)" <gi...@apache.org>]

tvalentyn commented on code in PR #31255:
URL: https://github.com/apache/beam/pull/31255#discussion_r1597730417


##########
sdks/python/apache_beam/ml/inference/onnx_inference_test.py:
##########
@@ -31,6 +31,10 @@
 from apache_beam.testing.util import assert_that
 from apache_beam.testing.util import equal_to
 
+if True:  # appease lint

Review Comment:
   ```suggestion
   if bool(1):  # lint doesn't like an unconditional `raise`.
   ```



-- 
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: github-unsubscribe@beam.apache.org

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