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

Add type annotations, part 2 #212

Merged
merged 13 commits into from
Sep 28, 2023

Conversation

andlaus
Copy link
Collaborator

@andlaus andlaus commented Sep 27, 2023

This is the second part of the quest for comprehensive type annotations. Besides the on-topic changes, it also contains a minor cleanup of the README, and the removal of the completely unused but confusingly named Parameter.get_coded_value() method.

Andreas Lauser <andreas.lauser@mercedes-benz.com>, on behalf of MBition GmbH.
Provider Information

andlaus and others added 11 commits September 27, 2023 15:48
- update table of contents
- remove duplicate License section

Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
there's still `.get_coded_value_as_bytes(encode_state)`, but that one
is most definitely used.

Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
for whatever reason, mypy did not complain about these last time...

Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
@andlaus andlaus requested a review from kayoub5 September 27, 2023 13:50
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@andlaus andlaus force-pushed the add_type_annotations_2 branch 3 times, most recently from 8e2ef5e to 9ca3ea5 Compare September 28, 2023 08:48
thanks to [at]kayoub5 for the suggestion!

Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
As usual the thanks go to [at]kayoub5!

Signed-off-by: Andreas Lauser <andreas.lauser@mbition.io>
Signed-off-by: Christian Hackenbeck <christian.hackenbeck@mbition.io>
Signed-off-by: Alexander Walz <alexander.walz@mbition.io>
@andlaus andlaus force-pushed the add_type_annotations_2 branch from 9ca3ea5 to cf061ff Compare September 28, 2023 09:07
@andlaus andlaus merged commit d3f4ce8 into mercedes-benz:main Sep 28, 2023
6 checks passed
@andlaus andlaus deleted the add_type_annotations_2 branch October 4, 2023 14:26
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