If there is an optional parameter and support SteamingResponse#7
If there is an optional parameter and support SteamingResponse#7xitowzys wants to merge 3 commits intodotX12:masterfrom
Conversation
xitowzys
changed the title
Fixed the test
| homepage = "https://github.com/xitowzys-ISZF/fastapi-gateway" | ||
| repository = "https://github.com/xitowzys-ISZF/fastapi-gateway" |
There was a problem hiding this comment.
Why was it changed from dotX12 to xitowzys-ISZF?
| [tool.poetry] | ||
| name = "fastapi_gateway" | ||
| version = "0.0.3" | ||
| name = "fastapi_gateway_ultra" |
There was a problem hiding this comment.
Also describe why you made the decision to change the name of the package and made a lot of edits about it, if it is appropriate and will bring some benefit, I can consider your suggestion, but for now it seems pointless to me.
|
And I think I understand, you made a fork and posted it on pypi under your own name, okay :) |
I apologize for the confusion that occurred here. I just want to continue developing this project further and was considering publishing it on PyPI with a different name under your fork. Also, I didn't realize that all subsequent changes would be reflected in Pull requests, as I'm still new to this process. |
|
No big deal, it's because u opened a PR from your master to my master branch, so it merges all changes here. |
Only now I noticed that. Thank you. I hope you will find time to review my previous changes 🙂. |
2 participants
When creating an optional parameter, it throws an error
TypeError: Invalid variable type: value should be str, int or float, got None of type <class 'NoneType'>Support SteamingResponse