-
Notifications
You must be signed in to change notification settings - Fork 599
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
refactor: replace ioutil=>io; update linter #1211
Conversation
Signed-off-by: Christopher Phillips <[email protected]>
remove deprecated linters that have been abandoned replace usage of ioutil => io/os upgrades Signed-off-by: Christopher Phillips <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM just one question about adding the removed linters to the comments
@@ -10,7 +10,6 @@ linters: | |||
enable: | |||
- asciicheck | |||
- bodyclose | |||
- deadcode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should the other linters be added to the commented section below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved the ones that were relevant to the generic issue. The ones that have been deleted are:
[WARN] x is deprecated. Replaced by unused.
I can include them commented out if we want to ever incorporate them again if they are picked up by a maintainer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah no need if they're deprecated 👍
Benchmark Test ResultsBenchmark results from the latest changes vs base branch
|
* main: refactor: replace ioutil=>io; update linter (#1211)
* main: bug: remove chance for panic; provide default attestation path (anchore#1214) refactor: update Makefile organization; update DEVELOPING.md instructions (anchore#1212) refactor: replace ioutil=>io; update linter (anchore#1211) Update bootstrap tools to latest versions. (anchore#1204) Add gosimports (anchore#1205) refactor: move formats from internal into syft module (anchore#1172) warn on errors from RPM DB parsing (anchore#1200) docs: improve Singularity image source docs (anchore#1190) Signed-off-by: Christopher Phillips <[email protected]>
Signed-off-by: Christopher Phillips <[email protected]>
Signed-off-by: Christopher Phillips <[email protected]>
Signed-off-by: Christopher Phillips <[email protected]>
Summary
Syft's
DEVELOPING.md
instructs the user to boostrap tools and runmake
The default make step has a couple of errors when running out of the box.
The first is
make lint
:These errors are correct by changing the directives at the top of these files to the newer
nolint:all
The second is where
io/util
is considered deprecated in the latest Golang release. These instances have been updated to their latestos
orio
equivlants.Abandoned linters have also been disabled per the warnings from the
golangci
tool:Changes
Follow up
make
does, and shorter introduction commands to get new users up to speedSigned-off-by: Christopher Phillips [email protected]