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
OneNote Bug Bounty - #177 #209
Conversation
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.
Thanks for the PR @p3rid0t. Let me know if you have questions about any of these comments.
I just pushed a commit with fixes to the comments mentioned here. Sorry it took a while, hope you don't mind. |
Pushed another commit with the small type tweaks mentioned in the comments :) |
Thanks @p3rid0t. I added another commit that applies some formatting fixes and a couple other tweaks to the method types that we were discussing. I think this looks good to me now. Thank you. |
This PR fixes issues mentioned in the bounty's issue - #177, aka:
As well as some general code improvements, and issue #164 related to line breaks and whitespace.
Looking forward to what needs to be improved or fixed. Thanks!