jrmccluskey commented on PR #23642: URL: https://github.com/apache/beam/pull/23642#issuecomment-1302302452
The Python PreCommit failures don't seem to be related to my code change, but everything else is sorted out. The linter didn't like the `Optional` tag on the `inference_fn` parameter since that maps the typing to `Callable | None` even with a default value, but since it's a trailing construction parameter with a default I don't think that's a huge change to remove it. -- 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]
