Skip to content

Conversation

@fenos
Copy link
Contributor

@fenos fenos commented Jan 28, 2026

What kind of change does this PR introduce?

Bug fix

What is the new behavior?

Return reply on healthcheck endpoints - prevent edge case crashes

Additional context

Add any other context or screenshots.

@coveralls
Copy link

coveralls commented Jan 28, 2026

Pull Request Test Coverage Report for Build 21448731198

Details

  • 1 of 16 (6.25%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.02%) to 75.944%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/http/routes/health/healthcheck.ts 0 2 0.0%
src/http/routes/admin/tenants.ts 0 3 0.0%
src/internal/database/migrations/migrate.ts 1 11 9.09%
Totals Coverage Status
Change from base Build 21443050779: -0.02%
Covered Lines: 25514
Relevant Lines: 33312

💛 - Coveralls

@fenos fenos force-pushed the fix/healthcheck-return-reply branch from b00125c to c7e462f Compare January 28, 2026 17:30
@fenos fenos merged commit 6c2355f into master Jan 28, 2026
2 checks passed
@fenos fenos deleted the fix/healthcheck-return-reply branch January 28, 2026 17:40
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.

4 participants