Skip to content
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

Provider Builder #10

Merged
merged 3 commits into from
Mar 18, 2024
Merged

Provider Builder #10

merged 3 commits into from
Mar 18, 2024

Conversation

techman83
Copy link
Contributor

Automatically instantiate providers into the stack as required, including the ability to inject config from the service settings.

@techman83 techman83 force-pushed the feat/providers branch 3 times, most recently from b176434 to 2ad050c Compare March 18, 2024 13:44
Automatically instantiate providers into the stack as required,
including the ability to inject config from the service settings.
There isn't a strong need to test a full json payload. This avoids issues when
the cdktf version changes
Dynaconf + cally providers have import errors for typing, this isn't
siginifcant for the purpose we're using.
Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
358 292 82% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/cally/cdk/_init_.py 58% 🟢
src/cally/cli/config/_init_.py 100% 🟢
src/cally/cli/config/loader.py 85% 🟢
TOTAL 81% 🟢

updated for commit: undefined by action🐍

@techman83 techman83 merged commit d226e47 into main Mar 18, 2024
3 checks passed
@techman83 techman83 deleted the feat/providers branch March 18, 2024 14:04
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