Skip to content

WIP: add a few type hints#1383

Draft
dcermak wants to merge 1 commit intoopenSUSE:masterfrom
dcermak:type-hints
Draft

WIP: add a few type hints#1383
dcermak wants to merge 1 commit intoopenSUSE:masterfrom
dcermak:type-hints

Conversation

@dcermak
Copy link
Copy Markdown
Member

@dcermak dcermak commented Aug 15, 2023

No description provided.

Comment thread osc/core.py
newinstance=False,
disable_build=False,
):
) -> Tuple[False, Optional[str], Optional[str], Optional[str], Optional[str]]: ...

Check notice

Code scanning / CodeQL

Statement has no effect Note

This statement has no effect.
Comment thread osc/core.py
maintenance=False,
newinstance=False,
disable_build=False,
) -> Union[Tuple[True, str, str, None, None], Tuple[False, Optional[str], Optional[str], Optional[str], Optional[str]]]: ...

Check notice

Code scanning / CodeQL

Statement has no effect Note

This statement has no effect.
@dmach
Copy link
Copy Markdown
Contributor

dmach commented Mar 11, 2026

I've rebased the pull request. If it looks fine to you, let's merge it.

@dcermak
Copy link
Copy Markdown
Member Author

dcermak commented Mar 11, 2026

I've rebased the pull request. If it looks fine to you, let's merge it.

It now broke all rpmbuilds because there's no typing_extensions available. I could change it to import from typing conditionally and add typing_extensions for old pythons

@dcermak dcermak force-pushed the type-hints branch 2 times, most recently from e8622aa to 227f63f Compare March 11, 2026 13:40
Comment thread osc/core.py
from io import StringIO
from pathlib import Path
from typing import Optional, Dict, Union, List, Iterable
from typing import TYPE_CHECKING, Any, Optional, Dict, Union, List, Iterable, Tuple, overload

Check notice

Code scanning / CodeQL

Unused import Note

Import of 'Any' is not used.
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.

3 participants