poetry init uses existing pyproject.toml if possible

[finswimmer]

At the moment a poetry init will fail, if a pyproject.toml already exists.

As this file is becoming more and more widely used to store configuration for other tools like black and isort, this PR changes the behavior to only skip creating the content for pyproject.toml if the file already contains a tool.poetry section or a build-system section.

closes: #1639

[abn]

Minor changes. Would be good to add test coverage for this.

[tarkatronic]

Instead of simply raising an error when a different build-system section already exists, would it be possible to prompt the user to convert it to use Poetry?

I recently ran into this when converting a project from setuptools to Poetry. In this case, I already had a pyproject.toml set up for a few other tools, and had a build-system defined. IIRC there can be cases where having a pyproject.toml without a build-system produces errors in pip, so I think this may not be an uncommon scenario.

[abn]

@finswimmer can we rebase this please?

[abn]

Looks good. We should add test coverage for this.

[abn]

@finswimmer here is an example for the init command test cases: abn@e378a88

[abn]

@finswimmer rebase please.

[finswimmer]

I'v expected this @abn :) Is done.

[abn]

Instead of simply raising an error when a different build-system section already exists, would it be possible to prompt the user to convert it to use Poetry?

@tarkatronic I thinnk that should be a new feature request as the scope of that is a bit larger than what is changed here.

[tarkatronic]

Instead of simply raising an error when a different build-system section already exists, would it be possible to prompt the user to convert it to use Poetry?

@tarkatronic I thinnk that should be a new feature request as the scope of that is a bit larger than what is changed here.

That is totally reasonable. 😄

[abn]

Looking good, minor changes.

[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.