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

fix: handle unset value for config.repo #72

Merged
merged 1 commit into from
Mar 8, 2023
Merged

Conversation

dnldsht
Copy link
Contributor

@dnldsht dnldsht commented Mar 8, 2023

resolve Cannot read properties of undefined (reading 'provider') if no repo is provided

resolve Cannot read properties of undefined (reading 'provider') if no repo is provided
@codecov
Copy link

codecov bot commented Mar 8, 2023

Codecov Report

Merging #72 (2e687d7) into main (a14d749) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   71.50%   71.50%           
=======================================
  Files           7        7           
  Lines         758      758           
  Branches       86       86           
=======================================
  Hits          542      542           
  Misses        216      216           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@pi0 pi0 changed the title fix: safe config.repo fix: handle undefined value for config.repo Mar 8, 2023
@pi0 pi0 changed the title fix: handle undefined value for config.repo fix: handle unset value for config.repo Mar 8, 2023
@pi0 pi0 merged commit a45d2c2 into unjs:main Mar 8, 2023
@pi0
Copy link
Member

pi0 commented Mar 8, 2023

Thanks!

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