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.mod from 1.16 to 1.18 #642
Conversation
yseto
commented
Aug 30, 2022
•
edited
edited
- go.mod from 1.16 to 1.18
- replace lint
- some fixes
SA1030: 'bitSize' argument is invalid, must be either 32 or 64 (staticcheck) ParseFloat will behave "64-bit" if given the wrong argument.
SA1006: printf-style function with dynamic format string and no further arguments should use print-style function instead (staticcheck)
@@ -41,7 +41,7 @@ func sizeValue(input string) (float64, error) { | |||
num := r[0][1] | |||
unit := r[0][2] | |||
|
|||
size, err = strconv.ParseFloat(num, 10) | |||
size, err = strconv.ParseFloat(num, 64) |
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.
👍
check-log/lib/check-log.go
Outdated
for _, entry := range entries { | ||
fi, err := entry.Info() | ||
if err == nil && detectInode(fi) == inode { | ||
return filepath.Join(dir, fi.Name()), nil | ||
} |
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.
Errors occurred in reading FileInfo will be ignored. Is it intended?
see fs.DirEntry
If the file has been removed or renamed
since the directory read, Info may return an error satisfying errors.Is(err, ErrNotExist).
Thanks for your review. |