-
Notifications
You must be signed in to change notification settings - Fork 798
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
split Azure packages into dedicated nuget packages #2038
Conversation
Hi @adamsitnik The idea is mantain the AzureStorage package with 3 references to Tables, Queues and Shares? Why not deprecated current Storage and move the IHealthCheckBuilder extensiosn to each package? |
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. @@ Coverage Diff @@
## master #2038 +/- ##
==========================================
- Coverage 69.06% 66.90% -2.17%
==========================================
Files 179 264 +85
Lines 4613 8641 +4028
Branches 447 620 +173
==========================================
+ Hits 3186 5781 +2595
- Misses 1323 2700 +1377
- Partials 104 160 +56
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@unaizorrilla the PR is ready for review, PTAL |
This PR contains following changes::
What is not included in this PR (I'll send a separate PR after this gets merged):
fixes #2040
fixes #1371 (health check reuse registered clients rather than creating new onesS)
fixes #1567 (client instance is created lazily when needed for the first time)
fixes #665 (two health checks were using the same name)