Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: allow throwing error with .response prop in upgrade #113

Merged
merged 4 commits into from
Jan 24, 2025

Conversation

LukeHagar
Copy link
Collaborator

No description provided.

@pi0 pi0 changed the title feat: proposing possible throw type solution feat: allow throwing error with .response prop Jan 24, 2025
@pi0 pi0 changed the title feat: allow throwing error with .response prop feat: allow throwing error with .response prop in upgrade Jan 24, 2025
@pi0
Copy link
Member

pi0 commented Jan 24, 2025

Thanks. Updated with getter, are you happy with it?

Copy link

codecov bot commented Jan 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@a1cab88). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #113   +/-   ##
=======================================
  Coverage        ?   76.55%           
=======================================
  Files           ?        9           
  Lines           ?      708           
  Branches        ?      146           
=======================================
  Hits            ?      542           
  Misses          ?      164           
  Partials        ?        2           

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

@LukeHagar
Copy link
Collaborator Author

Seems fine to me!

@pi0 pi0 merged commit 3bb5269 into unjs:main Jan 24, 2025
2 checks passed
@LukeHagar
Copy link
Collaborator Author

After publish I'll update the dep in the kit PR and test locally

@pi0
Copy link
Member

pi0 commented Jan 24, 2025

@LukeHagar LukeHagar deleted the throw-types branch January 24, 2025 21:07
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