-
Notifications
You must be signed in to change notification settings - Fork 808
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
Add file health check similar to folder health check #2027
Add file health check similar to folder health check #2027
Conversation
src/HealthChecks.System/DependencyInjection/SystemHealthCheckBuilderExtensions.cs
Outdated
Show resolved
Hide resolved
src/HealthChecks.System/DependencyInjection/SystemHealthCheckBuilderExtensions.cs
Outdated
Show resolved
Hide resolved
@michaelmairegger Thanks. I'm fine to merge after you add xml comments for new public APIs and some notes in README. |
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #2027 +/- ##
==========================================
+ Coverage 69.24% 69.35% +0.10%
==========================================
Files 176 178 +2
Lines 4582 4633 +51
Branches 445 452 +7
==========================================
+ Hits 3173 3213 +40
- Misses 1305 1313 +8
- Partials 104 107 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@sungam3r The xml comment is already existing. The new api for files is commented the same as the existing api for folders. |
This health check allows to check fo existing files similar to FolderHealthCheck