Skip to content

Add type annotations to a few files #904

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

Closed
wants to merge 2 commits into from
Closed

Conversation

qwergle
Copy link
Contributor

@qwergle qwergle commented Aug 7, 2021

This PR just adds some type annotations to a few files, and is meant to help with fixing #892. It's not much, but I'm still working on adding type annotations to all files.

@codecov-commenter
Copy link

codecov-commenter commented Aug 7, 2021

Codecov Report

Merging #904 (06d255a) into main (5eb31eb) will decrease coverage by 0.00%.
The diff coverage is 91.37%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #904      +/-   ##
==========================================
- Coverage   67.92%   67.92%   -0.01%     
==========================================
  Files          61       61              
  Lines        9147     9150       +3     
==========================================
+ Hits         6213     6215       +2     
- Misses       2934     2935       +1     
Impacted Files Coverage Δ
bpython/config.py 87.75% <ø> (ø)
bpython/curtsiesfrontend/repl.py 65.06% <ø> (ø)
bpython/repl.py 72.12% <ø> (-0.15%) ⬇️
bpython/test/test_autocomplete.py 94.84% <ø> (ø)
bpython/test/test_crashers.py 91.95% <ø> (ø)
bpython/urwid.py 0.00% <ø> (ø)
bpython/curtsies.py 51.72% <25.00%> (ø)
bpython/cli.py 15.13% <90.90%> (+0.07%) ⬆️
bpython/autocomplete.py 85.67% <97.22%> (+0.04%) ⬆️
bpython/_internal.py 78.57% <100.00%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5eb31eb...06d255a. Read the comment docs.

Copy link
Contributor

@sebastinas sebastinas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also run black to ensure consistent formatting after your changes.

@qwergle
Copy link
Contributor Author

qwergle commented Aug 13, 2021

The commit I just added performs all of the changes you requested, as well as reformatting many files using black.

@sebastinas
Copy link
Contributor

hmm, I had to rerun black again. Merged after doing that.

@sebastinas sebastinas closed this Aug 17, 2021
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