-
Notifications
You must be signed in to change notification settings - Fork 41
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
Dynamic Bridged Providers #1973
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1973 +/- ##
==========================================
- Coverage 61.36% 60.72% -0.64%
==========================================
Files 334 352 +18
Lines 44951 45494 +543
==========================================
+ Hits 27583 27628 +45
- Misses 15841 16342 +501
+ Partials 1527 1524 -3 ☔ View full report in Codecov by Sentry. |
99e477c
to
e54ade4
Compare
09a8275
to
f7964f1
Compare
Dropping to a lower level avoid cache corruption (which `EnsureProviderVersions` inexplicably allows) and allows us to avoid littering the working directory with local cache info.
This is simply more consistent.
Follow-up on dfda673.
7a7f115
to
70f5070
Compare
Initial PR for dynamically bridged providers.
Currently, support is limited to Parameterize and GetSchema calls. We are getting very close to where it would be helpful for multiple people to work on the project, so I'd like to get ready to merge soon.