Skip to content

Release/v1#36

Merged
rpmcginty merged 15 commits intomainfrom
release/v1
Mar 23, 2026
Merged

Release/v1#36
rpmcginty merged 15 commits intomainfrom
release/v1

Conversation

@rpmcginty
Copy link
Copy Markdown
Collaborator

These have been reviewed already.

…and syntax (typing.BUILTIN vs BUILTIN type hints)
- Updated request and response models across various modules to inherit from PydanticBaseModel.
- Replaced dataclass usage with Pydantic's field definitions for better validation and serialization.
- Adjusted S3URI references to S3Path in handler implementations for consistency.
- Enhanced type safety and validation in notification and data sync models.
…raise ValueError instead of AssertionError and update related tests
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 23, 2026

Codecov Report

❌ Patch coverage is 92.00000% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.79%. Comparing base (ec0582a) to head (c039bfe).
⚠️ Report is 16 commits behind head on main.

Files with missing lines Patch % Lines
src/aibs_informatics_aws_lambda/common/models.py 80.76% 3 Missing and 2 partials ⚠️
...matics_aws_lambda/handlers/data_sync/operations.py 83.33% 1 Missing and 2 partials ⚠️
...s_lambda/handlers/notifications/notifiers/model.py 86.36% 2 Missing and 1 partial ⚠️
...atics_aws_lambda/handlers/data_sync/file_system.py 77.77% 1 Missing and 1 partial ⚠️
...rmatics_aws_lambda/handlers/notifications/model.py 88.88% 1 Missing and 1 partial ⚠️
src/aibs_informatics_aws_lambda/main.py 75.00% 1 Missing and 1 partial ⚠️
...matics_aws_lambda/handlers/notifications/router.py 50.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   90.96%   90.79%   -0.18%     
==========================================
  Files          26       26              
  Lines        1428     1412      -16     
  Branches      122      131       +9     
==========================================
- Hits         1299     1282      -17     
+ Misses         98       94       -4     
- Partials       31       36       +5     
Files with missing lines Coverage Δ
.../aibs_informatics_aws_lambda/common/api/handler.py 85.29% <100.00%> (+0.14%) ⬆️
...aibs_informatics_aws_lambda/common/api/resolver.py 94.79% <100.00%> (+0.05%) ⬆️
src/aibs_informatics_aws_lambda/common/handler.py 94.87% <100.00%> (+0.08%) ⬆️
src/aibs_informatics_aws_lambda/common/logging.py 86.04% <100.00%> (-0.32%) ⬇️
src/aibs_informatics_aws_lambda/common/metrics.py 84.31% <100.00%> (-0.31%) ⬇️
...bs_informatics_aws_lambda/handlers/batch/create.py 83.78% <100.00%> (ø)
...ibs_informatics_aws_lambda/handlers/batch/model.py 100.00% <100.00%> (ø)
...informatics_aws_lambda/handlers/data_sync/model.py 100.00% <100.00%> (ø)
...tics_aws_lambda/handlers/demand/context_manager.py 96.61% <100.00%> (ø)
...bs_informatics_aws_lambda/handlers/demand/model.py 100.00% <100.00%> (ø)
... and 9 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rpmcginty rpmcginty merged commit 418bdd7 into main Mar 23, 2026
14 checks passed
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.

1 participant