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

[QMI-094] #95

Merged
merged 12 commits into from
Aug 14, 2024
Merged

[QMI-094] #95

merged 12 commits into from
Aug 14, 2024

Conversation

heevasti
Copy link
Collaborator

@heevasti heevasti commented Aug 9, 2024

Correcting and improving documentation across QMI transport related files. Also made some reformatting and typing changes, namely replacing Mapping with Dict when generic Mapping type is unnecessarily vague. And Mapping is imported now from abc.collections due to obsoletion in typing.

Correcting and improving documentation across QMI transport related files. Also made some reformatting and typing changes, namely replacing `Mapping` with `Dict` when generic `Mapping` type is unnecessarily vague. And `Mapping` is imported now from `abc.collections` due to obsoletion in `typing`.
@heevasti heevasti linked an issue Aug 9, 2024 that may be closed by this pull request
@heevasti heevasti requested a review from rbudhrani August 9, 2024 09:36
Badge Bot and others added 3 commits August 9, 2024 09:41
`Mapping` is imported now from `abc.collections` due to obsoletion in `typing` _for Python versions up from 3.9_.
…nsports' into 94-correct-documentation-for-transports
qmi/core/transport.py Outdated Show resolved Hide resolved
qmi/core/transport.py Show resolved Hide resolved
qmi/core/transport.py Outdated Show resolved Hide resolved
heevasti added 2 commits August 14, 2024 14:17
… to be generic key:value mapped types, and not to be changed (mutated).
@heevasti heevasti merged commit 0dd91da into main Aug 14, 2024
6 checks passed
@heevasti heevasti deleted the 94-correct-documentation-for-transports branch August 14, 2024 15:59
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.

Correct documentation for transports
2 participants