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: Handle empty repository autodiscovery #45

Merged
merged 1 commit into from
Jul 13, 2024
Merged

Conversation

meyfa
Copy link
Member

@meyfa meyfa commented Jul 13, 2024

When Renovate's autodiscovery turns up empty, for example when triggering a manual run with a non-existing repository scope, the log will not contain a list of matched repositories (not even an empty one). However, once the pod has completed, we know for a fact that no repositories are going to be discovered any more.
We can then show an empty list indicator, instead of continuing to show a loading indicator.

Screenshot:

image

When Renovate's autodiscovery turns up empty, for example when
triggering a manual run with a non-existing repository scope, the log
will not contain a list of matched repositories (not even an empty one).
However, once the pod has completed, we know for a fact that no
repositories are going to be discovered any more.
We can then show an empty list indicator, instead of continuing to
show a loading indicator,
@meyfa meyfa requested a review from a team as a code owner July 13, 2024 16:30
Copy link
Member

@lusu007 lusu007 left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@lusu007 lusu007 merged commit b35022a into main Jul 13, 2024
3 checks passed
@lusu007 lusu007 deleted the feat/empty-progress branch July 13, 2024 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants