MonkeyCanCode commented on code in PR #898: URL: https://github.com/apache/polaris/pull/898#discussion_r1947402992
########## regtests/client/python/pyproject.toml: ########## @@ -17,75 +17,37 @@ # under the License. # -[tool.poetry] +[project] name = "polaris" version = "1.0.0" description = "Apache Polaris Management Service" -authors = ["Apache Software Foundation <[email protected]>"] -license = "Apache License 2.0" +authors = [ + {name = "Apache Software Foundation", email = "[email protected]"} +] readme = "README.md" -repository = "https://github.com/apache/polaris/"; +requires-python = ">=3.9,<4.0" +license = "Apache-2.0" keywords = ["Polaris", "Polaris Management Service"] -include = ["polaris.management/py.typed"] -package-mode = true +dynamic = ["classifiers"] +dependencies = [ + "urllib3>=1.25.3,<2.0", + "python-dateutil>=2.8.2", + "pydantic>=2.0.0", + "typing-extensions>=4.7.1", + "boto3==1.36.10", +] -[tool.poetry.dependencies] -python = "^3.8" +[project.urls] +homepage = "https://polaris.apache.org/"; +repository = "https://github.com/apache/polaris/"; -urllib3 = "^1.25.3" -python-dateutil = ">=2.8.2" -pydantic = ">=2" -typing-extensions = ">=4.7.1" -boto3 = "==1.36.10" +[tool.poetry] +requires-poetry = ">=2.0" +packages = [{ include = "polaris" }] -[tool.poetry.dev-dependencies] +[tool.poetry.group.test.dependencies] pytest = ">=7.2.1" -flake8 = ">=4.0.0" -types-python-dateutil = ">=2.8.19.14" -mypy = "1.14.1" Review Comment: Yes, currently there is no pipeline for those linter/format as compared to what we are doing for the java code base. But I think we should introduce this in the next PR if no concern with this approach from the community. Then we should do the same enforcement for linter/format on CI pipeline. What do you think? @eric-maynard @collado-mike -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
