The new fastapi-cli is in circler dependency with the fastapi - this breaks bazel builds #11529
-
First Check
Commit to Help
Example Codefastapi==0.111.0 \
--hash=sha256:97ecbf994be0bcbdadedf88c3150252bed7b2087075ac99735403b1b76cc8fc0 \
--hash=sha256:b9db9dd147c91cb8b769f7183535773d8741dd46f9dc6676cd82eab510228cd7
# via
# fastapi-cli
# pact-python
fastapi-cli==0.0.2 \
--hash=sha256:589565ba758432632eadcf7b950e0ec76bb283b549784d9df17f261a8a9de476 \
--hash=sha256:d7a8ec89fd52ad16c52de9fe7299e4b22c7a32e1bf28aa886e5517e4927423dd
# via fastapi DescriptionBazel is very opinionated build system and does not allow for circler dependency. This unwise decision to create to libraries depending on each other will likely be breaking a lot of build when they attempt to update to the new version. Operating SystemLinux Operating System Detailsit does not matter FastAPI Version111 Pydantic Versiondoes not matter Python Versiondoes not matter Additional ContextNo response |
Beta Was this translation helpful? Give feedback.
Replies: 2 comments 3 replies
-
Doesn't even need fastapi-cli as a dep. Just fastapi breaks my builds. |
Beta Was this translation helpful? Give feedback.
-
Thanks for the feedback! I checked that Python indeed supported circular dependencies before setting it up, but seeing that Bazel (and maybe other systems) would handle dependencies in a more strict way, I removed the circular dependency requirement. 🤓 This is released in FastAPI CLI 0.0.4 🎉 |
Beta Was this translation helpful? Give feedback.
Thanks for the feedback! I checked that Python indeed supported circular dependencies before setting it up, but seeing that Bazel (and maybe other systems) would handle dependencies in a more strict way, I removed the circular dependency requirement. 🤓
This is released in FastAPI CLI 0.0.4 🎉