Skip to content

wip fix: unsupported types in request body#1243

Open
sansyrox wants to merge 2 commits intomainfrom
fix/unsupported-json-types
Open

wip fix: unsupported types in request body#1243
sansyrox wants to merge 2 commits intomainfrom
fix/unsupported-json-types

Conversation

@sansyrox
Copy link
Member

@sansyrox sansyrox commented Sep 6, 2025

Description

This PR fixes #

Summary

This PR does....

PR Checklist

Please ensure that:

  • The PR contains a descriptive title
  • The PR contains a descriptive summary of the changes
  • You build and test your changes before submitting a PR.
  • You have added relevant documentation
  • You have added relevant tests. We prefer integration tests wherever possible

Pre-Commit Instructions:

Review by RecurseML

🔍 Review performed on ffb7ebc..83e1ed5

Severity Location Issue Action
Low integration_tests/test_request_json.py:33 Redundant comment that restates what the code does Dismiss
Low integration_tests/test_request_json.py:50 Redundant comment that restates what the code does Dismiss
Low integration_tests/test_request_json.py:90 Redundant comment that restates what the code does Dismiss
Low integration_tests/test_request_json.py:92 Redundant comment that restates what the code does Dismiss
Low integration_tests/test_request_json.py:95 Redundant comment that restates what the code does Dismiss
✅ Files analyzed, no issues (2)

integration_tests/base_routes.py
src/types/request.rs

Analyze latest changes

Need help? Join our Discord

@vercel
Copy link

vercel bot commented Sep 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
robyn Error Error Sep 6, 2025 1:28pm

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 6, 2025

CodSpeed Performance Report

Merging #1243 will not alter performance

Comparing fix/unsupported-json-types (400aedb) with main (a77063a)1

Summary

✅ 150 untouched benchmarks

Footnotes

  1. No successful run was found on main (ffb7ebc) during the generation of this report, so a77063a was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Copy link

@recurseml recurseml bot left a comment

Choose a reason for hiding this comment

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

Review by RecurseML

🔍 Review performed on 83e1ed5..400aedb

✨ No bugs found, your code is sparkling clean

✅ Files analyzed, no issues (2)

integration_tests/test_request_json.py
integration_tests/base_routes.py

@sansyrox sansyrox force-pushed the main branch 3 times, most recently from 5fb08e0 to 0472e42 Compare February 15, 2026 06:39
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.

1 participant