-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
mypy: Fix return
issues
#8242
Comments
|
Oh dear. I can help and we do divide and conquer? |
I'll do a first pass, and see how time consuming it - then maybe we can divide some of the more complex areas. |
I've got it down to 54 errors. These are the files that are left: rasa/shared/nlu/training_data/message.py If you want to give me a hand, you could have a look at the first half? (i.e. to |
Sure 👍🏻 What's your branch? |
branch is |
I'm doing my changes on this PR #9054 (fixed |
Maxime recently enabled
mypy
support for the first couple of issues. In our current setup we ignore certain typing issues as there are currently too many to tackle them all at once.We should instead tackle them step by step. Related issue. Parent issue.
Current number of issues with this type: 18
Definition of Done:
mypy
checks forreturn
return
issues are fixedThe text was updated successfully, but these errors were encountered: