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.

Sign up for GitHub

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

Jump to bottom

type hints and api docs #20

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

type hints and api docs #20

wants to merge 3 commits into from

Conversation

sarnold
Copy link
Owner

@sarnold sarnold commented Nov 4, 2024

  • start adding type hints, update mypy config

@pep8speaks
Copy link

pep8speaks commented Nov 26, 2024 •
edited
Loading

Hello @sarnold! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-11-26 22:42:09 UTC

@sarnold
chg: dev: cleanup doc examples and script output
a250786 
* check for zero-length string in input IDs

Signed-off-by: Stephen L Arnold <[email protected]>
@github-actions GitHub Actions
Copy link

Package Line Rate Branch Rate Complexity Health
src.yaml_tools 97% 94% 0 ✔
Summary 97% (415 / 430) 94% (139 / 148) 0 ✔

@github-actions GitHub Actions
Copy link

Hello @sarnold! Thanks for opening this PR. We found the following information based on analysis of the coverage report:

Base Branch Rate coverage is 94%
Merging b674ba6 into main will not change coverage

Nice work, @sarnold. Cheers! 🍻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sarnold @pep8speaks