Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix typehints #182

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

fix typehints #182

wants to merge 4 commits into from

Conversation

JaskRendix
Copy link
Contributor

@JaskRendix JaskRendix commented Jan 18, 2024

PR reduces the number of typehints 289>149

While fixing, by following a chain of typehints, I arrived to the source here:

class TiledMap(TiledElement):
    def __init__(
        self,
        filename: Optional[str] = None, <---------- here

is it correct an Optional?

apps/pygame_sdl2_demo.py Outdated Show resolved Hide resolved
@JaskRendix JaskRendix marked this pull request as ready for review January 19, 2024 18:28
@JaskRendix JaskRendix force-pushed the typehints branch 6 times, most recently from 15849af to 320de11 Compare January 20, 2024 09:28
@JaskRendix JaskRendix marked this pull request as draft January 20, 2024 11:58
@JaskRendix JaskRendix force-pushed the typehints branch 4 times, most recently from 0745b18 to 1965ca6 Compare January 22, 2024 20:55
@JaskRendix JaskRendix force-pushed the typehints branch 3 times, most recently from 2c45582 to a41a47c Compare January 24, 2024 18:38
@JaskRendix JaskRendix marked this pull request as ready for review February 18, 2024 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants