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

feat!: remove deprecated CLI options #6210

Merged
merged 2 commits into from
Aug 22, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions docs/cli.md
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,6 @@ option is used.
* `--with`: The optional dependency groups to include.
* `--only`: The only dependency groups to include.
* `--only-root`: Install only the root project, exclude all dependencies.
* `--default`: Only include the main dependencies. (**Deprecated**)
* `--sync`: Synchronize the environment with the locked packages and the specified groups.
* `--no-root`: Do not install the root package (your project).
* `--dry-run`: Output the operations but do not execute anything (implicitly enables --verbose).
Expand Down Expand Up @@ -268,7 +267,6 @@ update the constraint, for example `^2.3`. You can do this using the `add` comma
* `--without`: The dependency groups to ignore.
* `--with`: The optional dependency groups to include.
* `--only`: The only dependency groups to include.
* `--default`: Only include the main dependencies. (**Deprecated**)
* `--dry-run` : Outputs the operations but will not execute anything (implicitly enables --verbose).
* `--no-dev` : Do not update the development dependencies. (**Deprecated**)
* `--lock` : Do not perform install (only update the lockfile).
Expand Down Expand Up @@ -471,7 +469,6 @@ required by
* `--why`: When showing the full list, or a `--tree` for a single package, display why a package is included.
* `--with`: The optional dependency groups to include.
* `--only`: The only dependency groups to include.
* `--default`: Only include the main dependencies. (**Deprecated**)
* `--no-dev`: Do not list the dev dependencies. (**Deprecated**)
mkniewallner marked this conversation as resolved.
Show resolved Hide resolved
* `--tree`: List the dependencies as a tree.
* `--latest (-l)`: Show the latest version.
Expand Down Expand Up @@ -688,7 +685,6 @@ group defined in `tool.poetry.dependencies` when used without specifying any opt
* `--without`: The dependency groups to ignore.
* `--with`: The optional dependency groups to include.
* `--only`: The only dependency groups to include.
* `--default`: Only include the main dependencies. (**Deprecated**)
* `--without-hashes`: Exclude hashes from the exported file.
* `--without-urls`: Exclude source repository urls from the exported file.
* `--with-credentials`: Include credentials for extra indices.
Expand Down
4 changes: 0 additions & 4 deletions src/poetry/console/application.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,10 +78,6 @@ def _load() -> Command:
"env list",
"env remove",
"env use",
# Plugin commands
"plugin add",
"plugin remove",
"plugin show",
# Self commands
"self add",
"self install",
Expand Down
6 changes: 0 additions & 6 deletions src/poetry/console/commands/group_command.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,6 @@ def _group_dependency_options() -> list[Option]:
flag=False,
multiple=True,
),
option(
"default",
None,
"Only include the main dependencies. (<warning>Deprecated</warning>)",
),
option(
"only",
None,
Expand Down Expand Up @@ -85,7 +80,6 @@ def activated_groups(self) -> set[str]:
}

for opt, new, group in [
("default", "only", MAIN_GROUP),
("no-dev", "only", MAIN_GROUP),
("dev", "with", "dev"),
]:
Expand Down
Empty file.
64 changes: 0 additions & 64 deletions src/poetry/console/commands/plugin/add.py

This file was deleted.

57 changes: 0 additions & 57 deletions src/poetry/console/commands/plugin/remove.py

This file was deleted.

32 changes: 0 additions & 32 deletions src/poetry/console/commands/plugin/show.py

This file was deleted.

Empty file.
31 changes: 0 additions & 31 deletions tests/console/commands/self/deprecated_plugin/test_add.py

This file was deleted.

56 changes: 0 additions & 56 deletions tests/console/commands/self/deprecated_plugin/test_remove.py

This file was deleted.

24 changes: 0 additions & 24 deletions tests/console/commands/self/deprecated_plugin/test_show.py

This file was deleted.

1 change: 0 additions & 1 deletion tests/console/commands/test_install.py
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@ def tester(
("--with bam --without bam", {MAIN_GROUP, "foo", "bar", "baz", "bim"}),
("--with foo --without foo", {MAIN_GROUP, "bar", "baz", "bim"}),
# deprecated options
("--default", {MAIN_GROUP}),
("--no-dev", {MAIN_GROUP}),
],
)
Expand Down
6 changes: 0 additions & 6 deletions tests/console/commands/test_show.py
Original file line number Diff line number Diff line change
Expand Up @@ -208,12 +208,6 @@ def _configure_project_with_groups(poetry: Poetry, installed: Repository) -> Non
f"--only {MAIN_GROUP}",
"""\
cachy 0.1.0 Cachy package
""",
),
(
"--default",
"""\
cachy 0.1.0 Cachy package
""",
),
(
Expand Down