-
Notifications
You must be signed in to change notification settings - Fork 2
Use Pydantic to model API requests/responses #77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
999cabe
updated server models for all responses
shreddd 14cd2cd
relative import
shreddd e9d1277
Add testing setup
shreddd eae2bb3
ruff
shreddd d06765e
Merge branch 'main' into pydantic-server
shreddd 16066fa
Update ci and skip failing tests
shreddd 68f02de
cleanup
shreddd b8a4c3e
Merge branch 'main' into pydantic-server
eecavanna 75320c7
Use standard `TestClient` instead of transitive `requests` package
eecavanna 084c978
Update GHA workflow to ingest data into `bertron_test` database
eecavanna 2589b52
Omit `-it` options from `docker compose run` in GHA workflow
eecavanna f02d6d7
Temporarily add "known passing" test to facilitate debugging GHA
eecavanna 2d6cb43
Update ingester to log the database name
eecavanna a6e0bac
Remove duplicate test used for debugging
eecavanna e7a73db
Patch the config object via both `import` paths
eecavanna 19c9cfd
Clarify comment
eecavanna 39ed799
Convert `pytest` into a `dev-dependency`
eecavanna 69341cc
Convert `httpx` into non-dev `dependency` because ingester imports it
eecavanna a6cab92
Refactor response type hints to get more validation from IDE
eecavanna ed03081
Fix entity id with `/`
shreddd c3db2bf
Validate Entity on ingest;
shreddd a016e86
update to use Entity object in ingest
shreddd 039ed63
add ingest-test target to simplify testing
shreddd 07c53be
Fix projection responses
shreddd 70787b8
Update docker-compose.yml
shreddd 7463251
comment
shreddd fe0e47b
ruff updates
shreddd 6676ae8
remove unused Dict
shreddd c759e8e
Omit `.venv` folder from volume mount to avoid host-guest interference
eecavanna 8241e8b
Update CONTRIBUTING.md
shreddd 89433de
Relocate model to `models.py`
eecavanna 5fcbce1
Merge branch 'pydantic-server' of https://github.com/ber-data/bertron…
eecavanna 264aa54
Use `ruff` to reformat Python source files (to resolve GHA failure)
eecavanna c757a87
Remove commented-out command for running ingest directly
eecavanna 98e0a82
Resolve type ambiguities in ingest script
eecavanna da01ccb
Combine two `startswith` calls into one using tuple syntax
eecavanna b610001
Clarify comment about what function does
eecavanna caf62e2
Add doctest and configure pytest to run it
eecavanna b631d10
Use `ruff` to reformat Python module (to resolve GHA failure)
eecavanna 35b1075
Re-indent command to reflect abstraction layers
eecavanna 8022e99
Run ingest script automatically via pytest fixture
eecavanna File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.