nutti / fake-bpy-module

Fake Blender Python API module collection for the code completion.
MIT License
1.37k stars 97 forks source link

Redundant `| None` types in the argument types #275

Closed Andrej730 closed 2 months ago

Andrej730 commented 3 months ago

On the left it's bpy/utils/__init__.py from fake-bpy, on the right - Blender 4.1 bpy/utils/__init__.py.

Note two issues: 1) path and create have a redundant | None in their types. 2) Signature is missing *, making path and create not keyword-only though they actually are.

image

Road-hog123 commented 3 months ago
  1. I guess the whole "if not Never None must accept None" should not be applied to parts of the API defined in Python files...

  2. I reported a similar issue in #266.

nutti commented 2 months ago

docstring of this function is also mistake. https://projects.blender.org/blender/blender/pulls/125316

nutti commented 2 months ago

I think this issue is different from #254 because this issue comes from the external module not Blender docs.

nutti commented 2 months ago

Fixed this issue now.