Skip to content

Conversation

@tleonhardt
Copy link
Member

Started adding optional type hinting.

This partially addresses #355.

@tleonhardt tleonhardt added this to the 0.9.0 milestone Apr 20, 2018
@tleonhardt tleonhardt self-assigned this Apr 20, 2018
@tleonhardt tleonhardt requested a review from kmvanbrunt April 20, 2018 06:59
@codecov
Copy link

codecov bot commented Apr 20, 2018

Codecov Report

Merging #363 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #363      +/-   ##
==========================================
- Coverage   91.24%   91.23%   -0.01%     
==========================================
  Files           3        3              
  Lines        2273     2271       -2     
==========================================
- Hits         2074     2072       -2     
  Misses        199      199
Impacted Files Coverage Δ
cmd2/rl_utils.py 100% <100%> (ø) ⬆️
cmd2/cmd2.py 91.26% <100%> (ø) ⬆️

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 58fdd08...da426b2. Read the comment docs.

@anselor
Copy link
Contributor

anselor commented Apr 20, 2018

Wow, I hope there was a script or PyCharm/Visual Studio Code function that did this automatically from the existing type hints in the comments.

Copy link
Contributor

@anselor anselor left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@tleonhardt
Copy link
Member Author

Unfortunately I did it manually. There are tools which will do it for you semi-automatically, but I didn't want to take the time to learn to use one of them.

@tleonhardt tleonhardt merged commit a0a46f9 into master Apr 20, 2018
@tleonhardt tleonhardt deleted the type_hints branch April 20, 2018 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants