Skip to content

Conversation

vincentsarago
Copy link
Member

@vincentsarago vincentsarago commented Oct 17, 2025

This PR does:

  • switch to pyproject.toml for package metadata
  • switch to UV
  • limit FastAPI to <0.118.0 to avoid 🔴 (the real issue might be fixed in another PR)

@@ -1,19 +0,0 @@
FROM python:3.12-slim
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't really need a docker image just for the docs

"stac_fastapi_extensions",
]
package = false

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

metadata for the whole repo (not a package)

exclude = ["tests/"]

[tool.hatch.build.targets.wheel]
only-include = ["stac_fastapi"]
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI: This is the important bit. we want the packages to be installed on a stac_fastapi/ directory (to keep the imports working). The hatch documentation is not 💯 clear about how to keep the namespace but this seems to be working.

I think we will release the package as an pre version to test it properly

ref: https://stackoverflow.com/a/77944019

@vincentsarago vincentsarago marked this pull request as ready for review October 20, 2025 10:40
Copy link
Collaborator

@hrodmn hrodmn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making the switch to uv! I think everything looks good but there might be one more change we need to make in scripts/publish.

@vincentsarago vincentsarago requested a review from hrodmn October 20, 2025 15:34
@vincentsarago
Copy link
Member Author

@jonhealy1 this PR might be of interest as well

Copy link
Collaborator

@jonhealy1 jonhealy1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super cool

@jonhealy1 jonhealy1 self-requested a review October 21, 2025 04:32
@vincentsarago vincentsarago merged commit a383f27 into main Oct 21, 2025
8 checks passed
@vincentsarago vincentsarago deleted the feature/switch-to-uv branch October 21, 2025 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants