Skip to content
This repository has been archived by the owner on Aug 12, 2024. It is now read-only.

Improve error message for all namespaces #914

Merged

Conversation

m1kola
Copy link
Member

@m1kola m1kola commented Jun 12, 2024

Will result in this message:

supported install modes [MultiNamespace OwnNamespace SingleNamespace] do not support target all namespaces

Instead of this when targeting all namespaces:

supported install modes [OwnNamespace SingleNamespace MultiNamespace] do not support target namespaces []

@m1kola m1kola requested a review from a team as a code owner June 12, 2024 13:27
Copy link

codecov bot commented Jun 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 14.44%. Comparing base (2cb02e5) to head (e12d038).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #914      +/-   ##
==========================================
+ Coverage   14.41%   14.44%   +0.03%     
==========================================
  Files          41       41              
  Lines        2553     2554       +1     
==========================================
+ Hits          368      369       +1     
  Misses       2128     2128              
  Partials       57       57              

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

@m1kola m1kola enabled auto-merge June 12, 2024 13:34
@m1kola m1kola added this pull request to the merge queue Jun 12, 2024
Merged via the queue into operator-framework:main with commit b5e3428 Jun 12, 2024
10 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants