-
Notifications
You must be signed in to change notification settings - Fork 3
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
Implement type hints #14
Conversation
@nickrsan I changed the exception messages, changed how the exceptions are being handled for the export_type check, and added type hintsfor pathlib Path objects |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Adam, nice work - you did great on everything - the things I had comments about were on things we hadn't discussed (primarily related to PyCharm reformatting the file). Are you willing to make the changes I mention in the comments, or would you prefer I do it myself?
@nickrsan Believe I have addressed everything. |
Wonderful. Thanks Adam. I'll take a look later tonight or tomorrow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great - really nice work and a good first PR on this repo. Thanks Adam.
Add type hints to all functions. There are still some parameters that need to be type hinted, but I am unsure of what they are supposed to be.