Skip to content

#813 "Use your github token" output is confusing to the user#814

Merged
knqyf263 merged 1 commit intoaquasecurity:mainfrom
paulmossman:813-Use-your-github-token-output-is-confusing
Jan 21, 2021
Merged

#813 "Use your github token" output is confusing to the user#814
knqyf263 merged 1 commit intoaquasecurity:mainfrom
paulmossman:813-Use-your-github-token-output-is-confusing

Conversation

@paulmossman
Copy link
Contributor

Changed the output string to "Using your github token".

@CLAassistant
Copy link

CLAassistant commented Jan 21, 2021

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented Jan 21, 2021

Codecov Report

Merging #814 (83eb902) into main (a00d719) will decrease coverage by 0.11%.
The diff coverage is 93.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #814      +/-   ##
==========================================
- Coverage   68.50%   68.39%   -0.12%     
==========================================
  Files          57       57              
  Lines        2210     2202       -8     
==========================================
- Hits         1514     1506       -8     
  Misses        564      564              
  Partials      132      132              
Impacted Files Coverage Δ
internal/app.go 98.38% <ø> (-0.10%) ⬇️
pkg/github/github.go 58.00% <0.00%> (ø)
pkg/detector/ospkg/redhat/redhat.go 93.54% <100.00%> (+0.21%) ⬆️
pkg/vulnerability/vulnerability.go 76.69% <100.00%> (+0.46%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6b29bf1...83eb902. Read the comment docs.

@knqyf263 knqyf263 merged commit 2acd1ca into aquasecurity:main Jan 21, 2021
@knqyf263
Copy link
Collaborator

Thanks!

@paulmossman
Copy link
Contributor Author

paulmossman commented Jan 21, 2021 via email

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