docutils: Correct 'parsers.rst.directives.format_values'

[stephenfin]

This helper function uses printf-style formatting to print output, therefore it shouldn't be limited to strings: calling e.g.

format_values(['a', 'b', None])

is perfectly reasonable.

Signed-off-by: Stephen Finucane [email protected]

[stephenfin]

This popped up in github.com/click-contrib/sphinx-click. To the best of my knowledge, you can call str(foo) for any type of foo and it will print something, so Any object seems the best option.

[srittau]

Thanks! But since – as you write – any value can be provided, confusingly object is the correct type, not Any. Any is the typing escape hatch, it basically means "we can't express this type using the current type system". While object means "we accept literally every possible value".

In this case, it makes no differences practically speaking, but we try to be correct here.

[stephenfin]

Thanks! But since – as you write – any value can be provided, confusingly object is the correct type, not Any. Any is the typing escape hatch, it basically means "we can't express this type using the current type system". While object means "we accept literally every possible value".

In this case, it makes no differences practically speaking, but we try to be correct here.

TIL! Will update 🙂

[github-actions]

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉