-
Notifications
You must be signed in to change notification settings - Fork 180
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
[DOC] Created a adding_typehints.md in developers_guide that Fixes issue #1857 #2424
Conversation
Added a documentation file on adding type hints that fixes aeon-toolkit#1857
Thank you for contributing to
|
@all-contributors please add @vedpawar2254 for Documentation |
I've put up a pull request to add @vedpawar2254! 🎉 |
Hi! Thanks for the PR, I'm wondering if that kind of content shouldn't be added as a section in the coding standard page rather than on its own. |
I think see where this ends up then decide |
I think this should be enforced as a standard to ensure consistent typing as tooling support is important to ensure code quality. |
Added a documentation file about adding type hints
Reference Issues/PRs
Fixes #1857
What does this implement/fix? Explain your changes.
This adds a file to the developers_guide and the file contains a guide to adding type hints in the project
Any other comments?
PR checklist
For all contributions