remove inaccurate comment
This convertor was initially implemented doing all the things the TODO says it doesn't yet do. The freestanding interpreter function is what doesn't do this.
This commit is contained in:
parent
d3804d0579
commit
1ff996ea67
|
@ -95,13 +95,8 @@ def _install_mode_validator(mode: T.List[T.Union[str, bool, int]]) -> T.Optional
|
|||
|
||||
|
||||
def _install_mode_convertor(mode: T.Optional[T.List[T.Union[str, bool, int]]]) -> FileMode:
|
||||
"""Convert the DSL form of the `install_mode` keyword argument to `FileMode`
|
||||
"""Convert the DSL form of the `install_mode` keyword argument to `FileMode`"""
|
||||
|
||||
This is not required, and if not required returns None
|
||||
|
||||
TODO: It's not clear to me why this needs to be None and not just return an
|
||||
empty FileMode.
|
||||
"""
|
||||
# this has already been validated by the validator
|
||||
return FileMode(*(m if isinstance(m, str) else None for m in mode))
|
||||
|
||||
|
|
Loading…
Reference in New Issue