use correct return type for c.html depending on input

[asmadsen]

I had some issues with using c.html with a non-promise body that had the return-type Response | Promise<Response>. This should use correct return-type depending on the input.

The author should do the following, if applicable

• bun run format:fix && bun run lint:fix to format the code

[yusukebe]

Hi @asmadsen Thank you for the PR.

Hey @usualoma Question. Can we think, if the first argument of c.html() is Promise, the return type is always Promise<Response>?

[usualoma]

Hi @asmadsen Thank you!

@yusukebe Yes, a Promise is returned whenever a Promise is given!

[asmadsen]

@yusukebe As @usualoma answered, if the first argument is Promise the return type is always Promise<Response>.

I saw there was a type error in the tests, so I fixed that. As well as added explicit type declarations so it would give an type error if the expected type is not returned.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.73%. Comparing base (614dff0) to head (66f588f).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2973   +/-   ##
=======================================
  Coverage   94.73%   94.73%           
=======================================
  Files         136      136           
  Lines       13335    13339    +4     
  Branches     2251     2229   -22     
=======================================
+ Hits        12633    12637    +4     
  Misses        702      702           

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

[yusukebe]

LGTM!

[yusukebe]

@asmadsen

Thank you for your contribution! I'll merge this now.