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

feat: add app health checks #9

Merged
merged 25 commits into from
Jan 31, 2024
Merged

Conversation

gulivero1773
Copy link
Contributor

@gulivero1773 gulivero1773 commented Jan 30, 2024

Implementation of application health check

This request fix issue - #7
Add structs for health check, add function for get version from string. Add data transfer object for health check, and add variables

Get health check of application

Copy link

codecov bot commented Jan 30, 2024

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

Copy link
Contributor

@mcmikius mcmikius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need resolve comments and fix Codecov warnings

Sources/AppHealthChecks/Response/ComponentName.swift Outdated Show resolved Hide resolved
Sources/AppHealthChecks/Response/ComponentType.swift Outdated Show resolved Hide resolved
Tests/AppHealthChecksTests/AppHealthChecksTests.swift Outdated Show resolved Hide resolved
Tests/AppHealthChecksTests/AppHealthChecksTests.swift Outdated Show resolved Hide resolved
Tests/AppHealthChecksTests/AppHealthChecksTests.swift Outdated Show resolved Hide resolved
.github/workflows/codeql.yml Outdated Show resolved Hide resolved
.github/workflows/test.yml Outdated Show resolved Hide resolved
@mcmikius mcmikius merged commit 2d4dca3 into main Jan 31, 2024
5 checks passed
@mcmikius mcmikius deleted the 7-add-application-health-check branch January 31, 2024 15:00
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.

2 participants