-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Refactor: Add more types and check step in the pipeline #297
Conversation
e8fe336
to
a5b31bb
Compare
8a198fb
to
b2fb44d
Compare
b2fb44d
to
6cc2427
Compare
Added more types. Unit tests work fine too. @codeacme17 @KillianLucas |
Hi @zya, I've read your changes, and I believe it's a great start for the refactoring project! |
@codeacme17 Thanks. Do you have approval or merge permissions? What is the process for getting this merged? |
@zya, sry bro, I am currently just a contributor to this project and don't have merge permissions. You might want to wait for an administrator to handle the merge. |
|
I personally think this step is a good prep for the refactor to begin. As there are not many tests, adding type hints before refactoring will help folks understand the code and refactor easier.
What do folks think?