Add --requirement option for pipx inject

CHANGELOG.md

@@ -14,6 +14,7 @@
 - [docs] Add subsection to make README easier to read
 - Add `pipx install --preinstall` to support preinstalling build requirements
 - Pass `--no-input` to pip when output is not piped to parent stdout
+- Add `--requirement` option for `pipx inject`
 
 ## 1.2.0
 

src/pipx/commands/inject.py

@@ -89,14 +89,32 @@ def inject(
     include_apps: bool,
     include_dependencies: bool,
     force: bool,
+    requirement_files: List[str],
 ) -> ExitCode:
     """Returns pipx exit code."""
     if not include_apps and include_dependencies:
         raise PipxError(
             "Cannot pass --include-deps if --include-apps is not passed as well"
         )
+    if not package_specs and not requirement_files:
+        raise PipxError(
+            "Dependencies or requirements files must be provided. See `pipx inject -h` for more info"
+        )
+
+    if package_specs and requirement_files:
+        raise PipxError(
+            "Dependencies and --requirement/-r option cannot be passed at the same time"
+        )
+
+    if requirement_files:
+        packages_list = []
+        for file in requirement_files:
+            with open(Path(file), "r") as f:
+                for line in f.readlines():
+                    packages_list.append(line.rstrip())
+
     all_success = True
-    for dep in package_specs:
+    for dep in package_specs or packages_list:
         all_success &= inject_dep(
             venv_dir,
             None,

src/pipx/main.py

@@ -222,6 +222,7 @@ def run_pipx_command(args: argparse.Namespace) -> ExitCode:  # noqa: C901
             None,
             args.dependencies,
             pip_args,
+            requirement_files=args.requirement,
             verbose=verbose,
             include_apps=args.include_apps,
             include_dependencies=args.include_deps,
@@ -374,8 +375,11 @@ def _add_inject(subparsers, venv_completer: VenvCompleter) -> None:
     ).completer = venv_completer
     p.add_argument(
         "dependencies",
-        nargs="+",
-        help="the packages to inject into the Virtual Environment--either package name or pip package spec",
+        nargs="*",
+        help=(
+            "The packages to inject into the Virtual Environment--either package name or pip package spec. "
+            "If --requirement or -r is passed, a requirements file is required instead of packages."
+        ),
     )
     p.add_argument(
         "--include-apps",
@@ -390,6 +394,16 @@ def _add_inject(subparsers, venv_completer: VenvCompleter) -> None:
         action="store_true",
         help="Modify existing virtual environment and files in PIPX_BIN_DIR",
     )
+    p.add_argument(
+        "--requirement",
+        "-r",
+        metavar="FILE",
+        action="append",
+        help=(
+            "Inject packages from the given requirements file. "
+            "Once this option is passed, dependencies passing from the command line will be omitted"
+        ),
+    )
     p.add_argument("--verbose", action="store_true")
 
 

tests/test_inject.py

@@ -1,3 +1,5 @@
+import textwrap
+
 import pytest  # type: ignore
 
 from helpers import mock_legacy_venv, run_pipx_cli
@@ -65,3 +67,23 @@ def test_inject_include_apps(pipx_temp_env, capsys, with_suffix):
             "--include-apps",
         ]
     )
+
+
+def test_inject_with_req_file(pipx_temp_env, capsys, tmp_path):
+    req_file = tmp_path / "requirements.txt"
+    req_file.write_text(
+        textwrap.dedent(
+            f"""
+                {PKG["black"]["spec"]}
+                {PKG["nox"]["spec"]}
+                {PKG["pylint"]["spec"]}
+            """
+        ).strip()
+    )
+    assert not run_pipx_cli(["install", "pycowsay"])
+    assert not run_pipx_cli(["inject", "pycowsay", "--requirement", str(req_file)])
+    assert run_pipx_cli(
+        ["inject", "pycowsay", PKG["black"]["spec"], "--requirement", str(req_file)]
+    )
+    captured = capsys.readouterr()
+    assert "cannot be passed at the same time" in captured.err