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

go/lint: only pass deps in our final binary to nancy #252

Merged
merged 1 commit into from
Apr 5, 2022

Conversation

adamdecaf
Copy link
Member

This extends on the work from #251 (and sonatype-nexus-community/nancy#228)

However, nancy requires the format of ".Path .Version" in the output, so we can force go list to output that.
See: golang/go#27900

@adamdecaf
Copy link
Member Author

This processes dependencies. I wish nancy would fail if it found zero deps to scan.

Screen Shot 2022-04-05 at 12 38 58 PM

@codecov-commenter
Copy link

Codecov Report

Merging #252 (c007bea) into master (cd899ca) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #252   +/-   ##
=======================================
  Coverage   78.37%   78.37%           
=======================================
  Files           1        1           
  Lines          37       37           
=======================================
  Hits           29       29           
  Misses          4        4           
  Partials        4        4           

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 cd899ca...c007bea. Read the comment docs.

@adamdecaf adamdecaf merged commit 5faf7a4 into moov-io:master Apr 5, 2022
@adamdecaf adamdecaf deleted the nancy-only-inspect-deps branch April 5, 2022 17:58
@darwinz
Copy link
Contributor

darwinz commented Apr 5, 2022

👍🏻

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