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

Add pooch cache manager verbosity control #482

Merged
merged 1 commit into from
Oct 2, 2023
Merged

Add pooch cache manager verbosity control #482

merged 1 commit into from
Oct 2, 2023

Conversation

bjlittle
Copy link
Owner

@bjlittle bjlittle commented Oct 2, 2023

🚀 Pull Request

Description

This pull-request adds explicit API level support to control the verbosity of the pooch cache manager within geovista.

Initially configured through the GEOVISTA_POOCH_MUTE environment variable (defaults to FALSE), thereafter through the geovista.cache.pooch_mute().


@github-actions github-actions bot added the type: testing Auto-labelled label Oct 2, 2023
@codecov
Copy link

codecov bot commented Oct 2, 2023

Codecov Report

Merging #482 (90b2e12) into main (ef9a0c9) will increase coverage by 0.05%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #482      +/-   ##
==========================================
+ Coverage   90.69%   90.74%   +0.05%     
==========================================
  Files          69       70       +1     
  Lines        3663     3684      +21     
==========================================
+ Hits         3322     3343      +21     
  Misses        341      341              
Files Coverage Δ
src/geovista/cache.py 85.71% <100.00%> (+1.26%) ⬆️
tests/cache/test_pooch_mute.py 100.00% <100.00%> (ø)

@bjlittle bjlittle merged commit 0dfaf17 into main Oct 2, 2023
16 checks passed
@bjlittle bjlittle deleted the pooch_mute branch October 2, 2023 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: testing Auto-labelled
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant