Skip to content

Conversation

@gadomski
Copy link
Member

@gadomski gadomski commented Apr 1, 2025

I copied the files from https://github.com/stapi-spec/stapi-fastapi/tree/main/src/stapi_fastapi/models and https://github.com/stapi-spec/stapi-fastapi/tree/main/src/stapi_fastapi/types and updated them to work. A couple of other choices and comments:

  • I switched to https://pypi.org/project/cql2/ as it's lighter than pygeofilter
  • I removed the callables from the pydantic models, they'll need to get re-added in a subclass (or something) in stapi-fastapi
  • I removed the distinction between "models" and "types"
  • Needs moar tests

@gadomski gadomski requested a review from jkeifer April 1, 2025 09:36
@gadomski gadomski self-assigned this Apr 1, 2025
@gadomski gadomski linked an issue Apr 1, 2025 that may be closed by this pull request
2 tasks
@gadomski gadomski linked an issue Apr 1, 2025 that may be closed by this pull request
@gadomski gadomski merged commit d26f690 into main Apr 1, 2025
1 check passed
@gadomski gadomski deleted the stapi-pydantic branch April 1, 2025 10:34
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.

Create stapi-pydantic Continuous Integration (CI)

2 participants