Skip to content

Conversation

ccojocar
Copy link
Member

@ccojocar ccojocar commented Oct 1, 2019

No description provided.

…nded in the packages docs

Signed-off-by: Cosmin Cojocar <[email protected]>
@ccojocar ccojocar requested a review from gcmurphy October 1, 2019 09:05
@codecov-io
Copy link

codecov-io commented Oct 1, 2019

Codecov Report

Merging #400 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #400   +/-   ##
=======================================
  Coverage   67.01%   67.01%           
=======================================
  Files           9        9           
  Lines         573      573           
=======================================
  Hits          384      384           
  Misses        170      170           
  Partials       19       19
Impacted Files Coverage Δ
analyzer.go 92.35% <100%> (ø) ⬆️

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 ad375d3...ddc9e1f. Read the comment docs.

Copy link
Member

@gcmurphy gcmurphy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems fine. Do we need to update supported go versions or anything like that for this?

@ccojocar
Copy link
Member Author

ccojocar commented Oct 2, 2019

It is available in all Go versions supported by gosec. It is defined anyhow in the packages package.

@ccojocar ccojocar merged commit e680875 into securego:master Oct 2, 2019
@ccojocar ccojocar deleted the replace-deprecated-loadmode branch May 13, 2024 16:17
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