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

Use the verbose on @info in fetch_all_apis_versions #62

Merged
merged 1 commit into from
Oct 24, 2023
Merged

Conversation

krynju
Copy link
Member

@krynju krynju commented Oct 24, 2023

this would avoid printing the logs on kuber init unless verbose is specified

julia> Kuber.set_api_versions!(ctx)
[ Info: unsupported events.k8s.io/v1
[ Info: unsupported autoscaling/v2
[ Info: unsupported certificates.k8s.io/v1
[ Info: unsupported policy/v1
[ Info: unsupported node.k8s.io/v1
[ Info: unsupported discovery.k8s.io/v1
[ Info: unsupported flowcontrol.apiserver.k8s.io/v1beta2
[ Info: unsupported helm.cattle.io/v1
[ Info: unsupported k3s.cattle.io/v1

@codecov-commenter
Copy link

Codecov Report

Merging #62 (e16caed) into master (fc1f428) will increase coverage by 1.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
+ Coverage   61.43%   62.55%   +1.12%     
==========================================
  Files           3        3              
  Lines         446      446              
==========================================
+ Hits          274      279       +5     
+ Misses        172      167       -5     
Files Coverage Δ
src/helpers.jl 82.27% <100.00%> (+1.81%) ⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@krynju krynju merged commit a9a838b into master Oct 24, 2023
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.

3 participants