Delete default groups #217
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dandavison
force-pushed
the
delete-default-groups
branch
from
ae8198e to
b21183a
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #190
Problem
The
uvx migrate-to-uvtool used in7a1dd4d623244a54170ec1b9952af84d8e5acc3ecreated adefault-groupssection, but this was not present previous in the poetry config and we don't think it's necessary, and in fact undesirable because it causesuv syncto pull in specialized dependencies. While this does add to the install time, the main way in which that's undesirable is that it make the project feel "bloated" -- people wishing to learn about core temporal functionality shouldn't need things like LLM SDKs or alternative Python concurrency frameworks.Solution
Delete the default groups section!