JOJ/JOJ3
25
6
Fork 0
Code Issues 4 Pull Requests Actions Packages Projects Releases Wiki Activity

feat: repo health check (#16) #17

Merged
张泊明518370910136 merged 37 commits from file_check into master 2024-09-11 20:09:27 +08:00
Conversation 20 Commits 37 Files Changed 22 +5 -3
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 48ab754ddf - Show all commits

8
pkg/healthcheck/verify.go
View File

@ -50,13 +50,15 @@ func VerifyFiles(rootDir string,checkFileNameList string, checkFileSumList strin
// rootDir := flag.String("rootDir", ".", "Root directory containing the files.")
// flag.Parse()
// Process input file names and checksums
if len(checkFileNameList) == 0 {
return nil//fmt.Errorf("No checksum happened")
os.Exit(1)
bomingzh marked this conversation as resolved Outdated
张泊明518370910136 commented 2024-09-10 16:14:23 +08:00
Outdated
Review
Copy Link

remove it

remove it
}
fileNames := strings.Split(checkFileNameList, ",")
checkSums := strings.Split(checkFileSumList, ",")
// Check if the number of files matches the number of checksums
if len(fileNames) == 0 {
return nil//fmt.Errorf("No checksum happened")
}
if len(fileNames) != len(checkSums) {
return fmt.Errorf("Error: The number of files and checksums do not match.")
张泊明518370910136 commented 2024-09-10 16:14:27 +08:00
Outdated
Review
Copy Link

remove it

remove it
张泊明518370910136 commented 2024-09-11 05:25:27 +08:00
Outdated
Review
Copy Link

I mean remove the os.Exit(1), and also the one on line 55.

I mean remove the `os.Exit(1)`, and also the one on line 55.