Skip to content

Conversation

@tpoliaw
Copy link
Contributor

@tpoliaw tpoliaw commented Nov 13, 2025

Graphql expects a POST request so warn when client uses GET

@tpoliaw tpoliaw changed the base branch from main to 404 November 13, 2025 10:15
@tpoliaw tpoliaw force-pushed the error-route-handlers branch 2 times, most recently from ec1da26 to 62c8c35 Compare November 13, 2025 10:17
@tpoliaw tpoliaw force-pushed the error-route-handlers branch from 62c8c35 to 11cc7ce Compare November 14, 2025 14:51
@tpoliaw tpoliaw force-pushed the error-route-handlers branch from 11cc7ce to 60df5d6 Compare November 14, 2025 15:09
@tpoliaw tpoliaw requested a review from abbiemery November 14, 2025 15:18
Base automatically changed from 404 to main November 14, 2025 16:24
@tpoliaw tpoliaw force-pushed the error-route-handlers branch 5 times, most recently from 29da6ed to 628db34 Compare November 18, 2025 11:05
@codecov
Copy link

codecov bot commented Nov 18, 2025

Codecov Report

❌ Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.36%. Comparing base (04ebb66) to head (6f955cf).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/main.rs 0.00% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
- Coverage   75.39%   74.36%   -1.03%     
==========================================
  Files          12       12              
  Lines         508      515       +7     
==========================================
  Hits          383      383              
- Misses        125      132       +7     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Graphql expects a POST request so warn when client uses GET.
@tpoliaw tpoliaw force-pushed the error-route-handlers branch from 628db34 to 6f955cf Compare November 21, 2025 14:50
@tpoliaw tpoliaw merged commit 71bb834 into main Nov 24, 2025
5 of 7 checks passed
@tpoliaw tpoliaw deleted the error-route-handlers branch November 24, 2025 17:21
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.

3 participants