这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@danielgtaylor
Copy link
Owner

This PR does the following things:

  1. Split the transform functionality out of API.Marshal(...) so that marshaling no longer needs access to the full request context.
  2. Better handle failures with transforming or marshaling responses after a response status code has already been sent on the wire.
  3. Rename test package from humahuma_test and dogfood using humatest now that there is no more circular dependency requiring yet another adapter implementation.

@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Files Coverage Δ
api.go 94.39% <100.00%> (+2.88%) ⬆️
huma.go 84.03% <100.00%> (+0.79%) ⬆️
error.go 96.42% <50.00%> (-3.58%) ⬇️

... and 3 files with indirect coverage changes

📢 Thoughts on this report? Let us know!.

@danielgtaylor danielgtaylor merged commit f0fa371 into main Oct 27, 2023
@danielgtaylor danielgtaylor deleted the dogfood-test branch October 27, 2023 21:03
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.

2 participants