-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/207 iterate existing objects #208
Merged
Merged
Conversation
This file contains 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
…ing down test colleciton)
chrisbc
added a commit
that referenced
this pull request
May 7, 2024
* poetry with pyproject requirements (#198) * updated pyproject requirements; smoketests and pytests OK; GHA script updates; * deploy using python3.10 and node20 * update serverless pacakges; fix yaml filename; * update aws-creds GHA version * fix aws secret names * add boto3 exclusion to serverless-requirements config * add library check module * add pyyaml to requirements * configire logs earlier; update aws logging config * Feature/195 elastic cli (#202) * remove pip test_requirements file * basic elastic_cli running * refactor script * improve cli for OHS analysis * configure black and isort, apply to scripts * format graphql_api/tests/* using black & isort * format graphql_api/* using black & isort; more doco; * Feature/207 iterate existing objects (#208) * rearrange base_data to avoid costly calls in __init__ (which was slowing down test colleciton) * WIP on object iteration * black & isort; * add more test cases * data store refactoring; * WIP on new resolver * WIP ; basic dynamodb object iteration working; * WIP ; S3 iteration basics; * S3 iteration working with pagination and FileData handled; * black & isort; testing OK * add reindex resolver; fix typo in cli logging config; * final housekeeping * add dynamodb:Query permission to lambda function; * fix type error for dynamodb iteration with ; * remove unneeded default id * handle after in dynamodb iterator
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
closing #207