Move some metadata from setup.cfg to pyproject.toml #426
Conversation
cocolato
force-pushed
the
feat/pyproject
branch
from
c203814 to
d66d8e4
Compare
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision d66d8e4 of this pull request into gerrit so we can run tests and reviews and stuff
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision d66d8e4 of this pull request into gerrit so we can run tests and reviews and stuff
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision d66d8e4 of this pull request into gerrit so we can run tests and reviews and stuff
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision d66d8e4 of this pull request into gerrit so we can run tests and reviews and stuff
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision d66d8e4 of this pull request into gerrit so we can run tests and reviews and stuff
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of zzzeek to try to get revision d66d8e4 of this pull request into gerrit so we can run tests and reviews and stuff
|
Patchset d66d8e4 added to existing Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/6102 |
|
sorry doing some config cahnges here |
There was a problem hiding this comment.
Federico Caselli (CaselIT) wrote:
code review left on gerrit
View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/6102
pyproject.toml
Outdated
| @@ -2,6 +2,66 @@ | |||
| build-backend = 'setuptools.build_meta' | |||
| requires = ['setuptools >= 47', 'wheel'] | |||
|
|
|||
| [project] | |||
| name = "Mako" | |||
| version = "1.3.11" | |||
There was a problem hiding this comment.
Federico Caselli (CaselIT) wrote:
this should be dynamic, since before it was attr: mako.__version__
See https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html#dynamic-metadata
View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/6102
| [project.urls] | ||
| Homepage = "https://www.makotemplates.org/" | ||
| Source = "https://github.com/sqlalchemy/mako" | ||
|
|
There was a problem hiding this comment.
Federico Caselli (CaselIT) wrote:
we miss Documentation = "https://docs.makotemplates.org"
View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/6102
|
Thanks! It has been updated. |
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of CaselIT to try to get revision 1ef7ade of this pull request into gerrit so we can run tests and reviews and stuff
|
Patchset 1ef7ade added to existing Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/6102 |
|
Ref: https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html#configuring-setuptools-using-pyproject-toml-files. Support for project.license-files and SPDX license expressions in project.license (PEP 639) were introduced in version 77.0.0. |
|
you can take a look at what's done in alembic or sqlalchemy, that both use pyproject in a fairly similar way:
it seems that both just require setuptools >=77.0.3 |
|
I've removed the |
There was a problem hiding this comment.
OK, this is sqla-tester setting up my work on behalf of CaselIT to try to get revision c52e72a of this pull request into gerrit so we can run tests and reviews and stuff
|
Patchset c52e72a added to existing Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/6102 |
Ref: https://peps.python.org/pep-0518/#sticking-with-setup-cfg
Fix: #420
Migrating the static matadata to project.toml will also help us to use some modern package managers such as: poetry、uv、pdm.