Skip to content
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

Combine poetry's utils and test groups #971

Open
gregtatum opened this issue Dec 20, 2024 · 0 comments
Open

Combine poetry's utils and test groups #971

gregtatum opened this issue Dec 20, 2024 · 0 comments
Labels
bug Something is broken or not correct

Comments

@gregtatum
Copy link
Member

Now that we have more dependencies across files, imports are easily missing. For instance a missing zstandard import now breaks utils unless you also install the test groups. Rather than duplicate the imports between the two groups, I would propose that we combine them together and just have a single general installation group.

Potential negatives are conflicting import requirements and bigger installs in CI.

If we don't want to do this, then at least we should fix it and write a CI test that runs the --help of every util to ensure that it's working.

@gregtatum gregtatum added the bug Something is broken or not correct label Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken or not correct
Projects
None yet
Development

No branches or pull requests

1 participant