[CHORE] Add maintainers, keywords, and URLs to pyproject#454
[CHORE] Add maintainers, keywords, and URLs to pyproject#454John McCall (lowlydba) wants to merge 1 commit intodevfrom
Conversation
Update multiple packages' pyproject.toml files to add a maintainers list, common keywords, and a [project.urls] section (Homepage, Source, Issues). Affects core, system, cli, annex, and all theme packages under packages/overture-schema-*. This centralizes package metadata to improve discoverability, attribution, and tooling integration.
|
Victor Schappert (@vcschapp) , Seth Fitzsimmons (@mojodna) - I didn't see an email on your GH profiles, so left that out, but happy to include it if you'd like to. |
There was a problem hiding this comment.
Pull request overview
Standardizes Python package metadata across the packages/* workspace modules to improve PyPI/project discoverability and maintenance (maintainers, keywords, and project URLs).
Changes:
- Added
maintainersmetadata across packagepyproject.tomlfiles. - Added
keywordsmetadata across packagepyproject.tomlfiles. - Added
[project.urls](Homepage/Source/Issues) to most packagepyproject.tomlfiles.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/overture-schema/pyproject.toml | Adds maintainers/keywords for the umbrella package (note: URLs not added here in this PR). |
| packages/overture-schema-transportation-theme/pyproject.toml | Adds maintainers/keywords and [project.urls]. |
| packages/overture-schema-system/pyproject.toml | Adds maintainers/keywords and [project.urls]. |
| packages/overture-schema-places-theme/pyproject.toml | Adds maintainers/keywords and [project.urls]. |
| packages/overture-schema-divisions-theme/pyproject.toml | Adds maintainers/keywords and [project.urls]. |
| packages/overture-schema-core/pyproject.toml | Adds maintainers/keywords and [project.urls]. |
| packages/overture-schema-cli/pyproject.toml | Adds maintainers/keywords (incl. cli) and [project.urls]. |
| packages/overture-schema-buildings-theme/pyproject.toml | Adds maintainers/keywords and [project.urls]. |
| packages/overture-schema-base-theme/pyproject.toml | Adds maintainers/keywords and [project.urls]. |
| packages/overture-schema-annex/pyproject.toml | Adds maintainers/keywords and [project.urls] (placed after [tool.uv.sources]). |
| packages/overture-schema-addresses-theme/pyproject.toml | Adds maintainers/keywords and [project.urls]. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Jennings Anderson (jenningsanderson)
left a comment
There was a problem hiding this comment.
Is there a way we could define a maintainers.yml or something and reuse it?
Or generally put a generic "Overture" as the maintainer? I'm not sure who the right maintainer is here.
| {name = "Vic Schappert"}, | ||
| {name = "Jennings Anderson", email = "jenningsa@meta.com"}, | ||
| ] | ||
| keywords = ["schema", "pydantic", "overture", "geospatial", "validation"] |
There was a problem hiding this comment.
There's a chance we're going to use this field for extension discovery (still being debated), so I'd rather omit this (for PyPI purposes).
There was a problem hiding this comment.
No problem, will get those removed
| maintainers = [ | ||
| {name = "Dana Bauer", email = "dana@overturemaps.org"}, | ||
| {name = "Seth Fitzsimmons"}, | ||
| {name = "Vic Schappert"}, | ||
| {name = "Jennings Anderson", email = "jenningsa@meta.com"}, | ||
| ] |
There was a problem hiding this comment.
+1 to Jennings Anderson (@jenningsanderson)
I'm unclear who the right maintainers are for this project. It's the Schema WG more broadly.
Yeah, generic Overture is totally fine - no pressure for anyone. |
This pull request standardizes and improves the project metadata across all Python package modules in the repository.
The main changes are:
These changes collectively make the packages more maintainable and user-friendly for both contributors and end users.
Partially resolves https://github.com/OvertureMaps/ops-team/issues/258