Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@samspills
Copy link
Contributor

fixed some conflicts, the two things I'd like someone to review to confirm are:

  1. uri suite tests, I left it as Seq() instead of Nil because of the conversation about type inference? But I'm not sure when/why the switch to Nil in the tests, maybe I should have kept it. Let me know!
  2. response logger, I moved error logger inside the impl

@rossabaker
Copy link
Member

I can't figure out how to review on the GitHub app. I managed to review the commit in the few pixels that aren't covered in Copilot buttons.

👍 on green

@samspills samspills merged commit 42d6410 into main Sep 23, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants