Conversation
| { name = "Nicola Coretti", email = "[email protected]" }, | ||
| { name = "Ariel Schulz", email = "[email protected]" }, | ||
| ] | ||
| requires-python = ">=3.10,<4.0" |
There was a problem hiding this comment.
✅ re-orders section, I think consistently - could check docu
| "infrastructure", | ||
| ] | ||
| dynamic = ["dependencies"] | ||
| dependencies = [ |
There was a problem hiding this comment.
✅ does correctly and orders
| "exasol", | ||
| "infrastructure", | ||
| ] | ||
| dynamic = ["dependencies"] |
There was a problem hiding this comment.
➖ needed to remove this manually, as it remains
| "twine>=6.1.0,<7", | ||
| ] | ||
|
|
||
| [project.urls] |
There was a problem hiding this comment.
originally this was incorrectly labeled, so it didn't get converted.
neither poetry nor uv were angry about that.
| Issues = "https://github.com/exasol/python-toolbox/issues" | ||
| Changelog = "https://exasol.github.io/python-toolbox/changelog.html" | ||
|
|
||
| [project.scripts] |
There was a problem hiding this comment.
✅ correctly converted & works in CLI (after creating new env)
| tbx = "exasol.toolbox.tools.tbx:CLI" | ||
| sphinx-multiversion = "exasol.toolbox.sphinx.multiversion:main" | ||
|
|
||
| # unused entry point as using project.scripts instead |
There was a problem hiding this comment.
➖ kept/added this and it should be manually removed
| # manually remove | ||
| #[project.entry-points] | ||
|
|
||
| [dependency-groups] |
There was a problem hiding this comment.
✅ did this correctly and ordered it
pyproject.toml
Outdated
| "cookiecutter>=2.6.0,<3", | ||
| ] | ||
|
|
||
| # Needed for poetry intsall to run again |
There was a problem hiding this comment.
➖ I don't think there's a way to preserve poetry too much, so this needed to be manually copied back over these two sections (see intermediate commits).
| sphinx-inline-tabs = "^2023.4.21" | ||
| sphinx-design = ">=0.5.0,<1" | ||
| sphinx-toolbox = "^4.0.0" | ||
| typer = { extras = ["all"], version = ">=0.7.0" } |
There was a problem hiding this comment.
✅ uv told us typer[all] isn't a thing, but ➖ we needed to manually do
078a17f to
f5233d3
Compare
| sphinx-multiversion = 'exasol.toolbox.sphinx.multiversion:main' | ||
|
|
||
| # manually re-add as needed for code | ||
| [tool.poetry.requires-plugins] |
There was a problem hiding this comment.
➖ need to manually re-add
| "F", # Pyflakes rules (excluding F401) | ||
| "UP", # pyupgrade rules | ||
| "D", # Docstring rules | ||
| "E", # Syntax errors |
There was a problem hiding this comment.
Sometimes it removes comments. Here it kept them, but this would be something to watch out for.
| "cookiecutter>=2.6.0,<3", | ||
| ] | ||
|
|
||
| # Needed for poetry install to run again |
There was a problem hiding this comment.
➖ needed to override these two sections with what we had before
| { name = "Nicola Coretti", email = "[email protected]" }, | ||
| { name = "Ariel Schulz", email = "[email protected]" }, | ||
| ] | ||
| requires-python = ">=3.10,<4.0" |
There was a problem hiding this comment.
Note that this migration mostly works for PTB as most dependencies are main ones but would need to switch to Poetry 2.2.x or update PTB otherwise to handle dev dependencies moving to groups.
| result = pyproject_toml.get_section_dict("test") | ||
| assert result is None | ||
|
|
||
| @staticmethod |
There was a problem hiding this comment.
This one was initially failing after the pyproject.toml update as the sections are hard-coded in exasol/toolbox/util/dependencies/poetry_dependencies.py. If we wanted to initially maintain the old and new versions, we should create a hard-coded pyproject.toml that looks like the oldway and adapt one or more tests for that and create a follow issue to remove thisold` code & tests.
|
|
||
| # Needed for poetry install to run again | ||
| [tool.poetry] | ||
| requires-poetry = ">=2.1.0" |
There was a problem hiding this comment.
This should be bumped
| requires-poetry = ">=2.1.0" | |
| requires-poetry = ">=2.2.1" |
| # Needed for poetry install to run again | ||
| [tool.poetry] | ||
| requires-poetry = ">=2.1.0" | ||
| packages = [ |
There was a problem hiding this comment.
The cookie-cutter template also needs to be adjusted to be PEP-compliant and have poetry >= 2.2.1
Checklist
Note: If any of the items in the checklist are not relevant to your PR, just check the box.
For any Pull Request
Is the following correct:
When Changes Were Made
Did you:
When Preparing a Release
Have you: