install-poetry: prompt before deleting data_dir

[br3ndonland]

The install-poetry.py script supports an --uninstall option. The uninstall performs shutil.rmtree(str(self._data_dir)).

poetry/install-poetry.py

Lines 479 to 500 in 82459bd

	def uninstall(self) -> int:
	if not self._data_dir.exists():
	self._write(
	"{} is not currently installed.".format(colorize("info", "Poetry"))
	)
	return 1
	version = None
	if self._data_dir.joinpath("VERSION").exists():
	version = self._data_dir.joinpath("VERSION").read_text().strip()
	if version:
	self._write(
	"Removing {} ({})".format(
	colorize("info", "Poetry"), colorize("b", version)
	)
	)
	else:
	self._write("Removing {}".format(colorize("info", "Poetry")))
	shutil.rmtree(str(self._data_dir))

If $POETRY_HOME is set, self._data_dir is set to $POETRY_HOME.

poetry/install-poetry.py

Line 395 in 82459bd

self._data_dir = data_dir()

poetry/install-poetry.py

Lines 134 to 136 in 82459bd

	def data_dir(version: Optional[str] = None) -> Path:
	if os.getenv("POETRY_HOME"):
	return Path(os.getenv("POETRY_HOME")).expanduser()

Uninstallation will therefore delete the entire $POETRY_HOME directory.

This PR will update install-poetry.py to prompt the user before performing this destructive operation.

[neersighted]

I'm thinking this is out of scope for two reasons. The first is the issue of setting $POETRY_HOME=~/.local was never intended, and the POETRY_HOME directory was meant to be managed entirely by Poetry. The docs could likely be improved on this (e.g. if you put anything else in that directory, it may be destroyed), but this is not out of line with most developer tools.

Secondly, the installer script was recently migrated to https://github.com/python-poetry/install.python-poetry.org. I would suggest opening an issue/PR there if you still feel this change is important.

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.