Closed tonglil closed 3 months ago
Attention: Patch coverage is 41.17647%
with 10 lines
in your changes missing coverage. Please review.
Project coverage is 71.05%. Comparing base (
6b528bc
) to head (dada752
). Report is 94 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
pkg/vault/client.go | 41.17% | 5 Missing and 5 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
good job! I will add one commit for updating the docs and then merge and release it along with the other PR
Instead of checking for env vars, prefer the standard configuration and check for the use of a tokenhelper (ie existing token at
~/.vault-token
).