Skip to content

fix: show info message for intentional panic routes#145

Merged
Xendergo merged 1 commit into
masterfrom
intpanic
Jun 14, 2026
Merged

fix: show info message for intentional panic routes#145
Xendergo merged 1 commit into
masterfrom
intpanic

Conversation

@ericswpark

Copy link
Copy Markdown
Member

This adds a little message for our intentional panicking route so that it reassures sysadmins that monitor our server logs

Copilot AI review requested due to automatic review settings June 14, 2026 01:55
@ericswpark ericswpark self-assigned this Jun 14, 2026
@ericswpark ericswpark requested review from Xendergo, Copilot and kdkasad and removed request for Copilot June 14, 2026 01:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the intentional panic demo route to include a clearer panic message, making the resulting panic output easier to interpret when reviewing logs and the rendered error page.

Changes:

  • Add a descriptive panic payload for the /debug/panic route to indicate the panic is intentional.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov

codecov Bot commented Jun 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 87.86%. Comparing base (e1a7b83) to head (7c401c8).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/routes.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #145   +/-   ##
=======================================
  Coverage   87.86%   87.86%           
=======================================
  Files           8        8           
  Lines        2430     2430           
  Branches     2430     2430           
=======================================
  Hits         2135     2135           
  Misses        261      261           
  Partials       34       34           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ArhanChaudhary

Copy link
Copy Markdown
Member

LGTM!

@github-actions

Copy link
Copy Markdown

Test Results

46 tests   46 ✅  2s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 7c401c8.

@ericswpark

Copy link
Copy Markdown
Member Author

@ArhanChaudhary you didn't actually leave a review 😤

@Xendergo Xendergo merged commit 5b87efd into master Jun 14, 2026
17 of 18 checks passed
@Xendergo Xendergo deleted the intpanic branch June 14, 2026 02:43
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