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.

Sign up for GitHub

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

Jump to bottom

Treat default dependencies as main group #1785

Merged
merged 3 commits into from
Oct 28, 2024
Merged

Treat default dependencies as main group #1785

merged 3 commits into from
Oct 28, 2024

Commits on Oct 28, 2024

  1. feat: treat default dependencies as main group 

    Poetry documentation on dependency groups [1] says:
  "The dependencies declared in tool.poetry.dependencies are part of an
implicit main group."

The current implementation does not account for that and in particular
does not allow to skip installation of default dependencies like in one
of
`poetry install --without main`
`poetry install --only-root`
`poetry install --only othergroup`.

This commit adds main as an implicit group and also adds it as a default
value where applicable.

[1]
https://python-poetry.org/docs/managing-dependencies#dependency-groups

Closes nix-community#1578
    @Lykos153 @cpcloud
    Lykos153 authored and cpcloud committed Oct 28, 2024
    Configuration menu
    Copy the full SHA
    a269d29 View commit details
    Browse the repository at this point in the history

  2. chore: handle the case where the group does not exist

    @cpcloud
    cpcloud committed Oct 28, 2024
    Configuration menu
    Copy the full SHA
    0e1f659 View commit details
    Browse the repository at this point in the history

  3. style: format default.nix

    @cpcloud
    cpcloud committed Oct 28, 2024
    Configuration menu
    Copy the full SHA
    61889c3 View commit details
    Browse the repository at this point in the history