[ 
https://issues.apache.org/jira/browse/BEAM-9012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17011517#comment-17011517
 ] 

Kenneth Knowles commented on BEAM-9012:
---------------------------------------

Peanut gallery: Seems like satisfying both static checkers is worth it in this 
case. I'm employed by Google, but my heart belongs to types/verification, so 
I'm not just saying this because of my paycheck.

> Include `-> None` on Pipeline and PipelineOptions `__init__` methods for 
> pytype compatibility
> ---------------------------------------------------------------------------------------------
>
>                 Key: BEAM-9012
>                 URL: https://issues.apache.org/jira/browse/BEAM-9012
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-py-core
>            Reporter: Brian Hulette
>            Assignee: Brian Hulette
>            Priority: Major
>             Fix For: 2.19.0
>
>          Time Spent: 1h 10m
>  Remaining Estimate: 0h
>
> mypy [made a decision|https://github.com/python/mypy/issues/604] to allow 
> init methods to omit {{\-> None}} return type annotations, but pytype has no 
> such feature. I think we should include {{\-> None}} annotations for pytype 
> compatibility.
> cc: [~chadrik]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to