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 issue where charts list errors if only helm hub registered #493

Merged
merged 2 commits into from
Sep 21, 2020

Conversation

richard-cox
Copy link
Member

  • this was a regression
  • we shouldn't make the call if there's no registered endpoints of that type anyway

@richard-cox richard-cox added the ready for review Ready for review label Sep 18, 2020
@richard-cox richard-cox self-assigned this Sep 18, 2020
@richard-cox richard-cox mentioned this pull request Sep 18, 2020
@richard-cox richard-cox changed the title Fix issue where charts list errored if both helm hub and helm repo's are registered Fix issue where charts list errored if only helm hub registered Sep 18, 2020
…are registered

- this was a regression
- we shouldn't make the call if there's no registered endpoints anyway
@richard-cox richard-cox force-pushed the fix-helm-hub-only-chart-list branch from 6dcb0ef to f6b4192 Compare September 18, 2020 13:31
- apply flex to ensure form fields are correct width
@codecov-commenter
Copy link

Codecov Report

Merging #493 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #493   +/-   ##
=======================================
  Coverage   57.83%   57.83%           
=======================================
  Files        1064     1064           
  Lines       34393    34393           
  Branches     4876     4876           
=======================================
  Hits        19892    19892           
  Misses      14501    14501           

@richard-cox richard-cox changed the title Fix issue where charts list errored if only helm hub registered Fix issue where charts list errors if only helm hub registered Sep 18, 2020
Copy link
Member

@nwmac nwmac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nwmac nwmac merged commit a95b49b into master Sep 21, 2020
@nwmac nwmac deleted the fix-helm-hub-only-chart-list branch September 21, 2020 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review Ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants