Remove --unpin

Author: dukecat0

changelog.d/1611.bugfix.md

@@ -1 +1 @@
-`pipx reinstall`: An exception will now be raised if package is pinned. `--unpin` option is introduced to allow reinstalling pinned package
+`pipx reinstall`: An exception will now be raised if package is pinned.

src/pipx/commands/reinstall.py

@@ -27,7 +27,6 @@ def reinstall(
     verbose: bool,
     force_reinstall_shared_libs: bool = False,
     python_flag_passed: bool = False,
-    unpin: bool = False,
 ) -> ExitCode:
     """Returns pipx exit code."""
     if not venv_dir.exists():
@@ -55,8 +54,8 @@ def reinstall(
     else:
         package_or_url = venv.main_package_name
 
-    if venv.pipx_metadata.main_package.pinned and not unpin:
-        raise PipxError(f"{error} Package {venv_dir} is pinned. Pass --unpin to unpin and reinstall it.")
+    if venv.pipx_metadata.main_package.pinned:
+        raise PipxError(f"{error} Package {venv_dir} is pinned. Run `pipx unpin {venv_dir.name}` to unpin it first.")
 
     uninstall(venv_dir, local_bin_dir, local_man_dir, verbose)
 

src/pipx/main.py

@@ -397,7 +397,6 @@ def run_pipx_command(args: argparse.Namespace, subparsers: Dict[str, argparse.Ar
             python=args.python,
             verbose=verbose,
             python_flag_passed=python_flag_passed,
-            unpin=args.unpin,
         )
     elif args.command == "reinstall-all":
         return commands.reinstall_all(
@@ -732,7 +731,6 @@ def _add_reinstall(subparsers, venv_completer: VenvCompleter, shared_parser: arg
         parents=[shared_parser],
     )
     p.add_argument("package").completer = venv_completer
-    p.add_argument("--unpin", action="store_true", help="Unpin and reinstall the package if it is pinned.")
     add_python_options(p)