feat/reposize #56

Merged
张泊明518370910136 merged 8 commits from feat/reposize into master 2024-10-12 18:30:55 +08:00
2 changed files with 5 additions and 4 deletions
Showing only changes of commit 8fb707e07d - Show all commits

View File

@ -44,6 +44,7 @@ func main() {
checkRelease := flag.Bool("checkRelease", true, "trigger release check")
rootDir := flag.String("root", "", "")
repo := flag.String("repo", "", "")
size := flag.Int("reposize", 2, "size of the repo")
localList := flag.String("localList", "", "")
droneBranch := flag.String("droneBranch", "", "")
checkFileNameList := flag.String("checkFileNameList", "", "Comma-separated list of files to check.")
@ -57,7 +58,7 @@ func main() {
}
setupSlog()
var err error
err = healthcheck.RepoSize()
err = healthcheck.RepoSize(*size)
if err != nil {
fmt.Printf("### Repo Size Check Failed:\n%s\n", err.Error())
}

View File

@ -10,7 +10,7 @@ import (
// RepoSize checks the size of the repository to determine if it is oversized.
// It executes the 'git count-objects -v' command to obtain the size information,
func RepoSize() error {
func RepoSize(conf_size int) error {
zzjc123 marked this conversation as resolved Outdated

We use camel case for variables in Go project.

We use camel case for variables in Go project.
// TODO: reimplement here when go-git is available
// https://github.com/go-git/go-git/blob/master/COMPATIBILITY.md
cmd := exec.Command("git", "count-objects", "-v")
@ -33,8 +33,8 @@ func RepoSize() error {
sum += size
}
}
if sum > 2048 {
return fmt.Errorf("Repository larger than 2MB. Please clean up or contact the teaching team.")
if sum > conf_size {
return fmt.Errorf("Repository larger than %d M. Please clean up or contact the teaching team.", conf_size)
zzjc123 marked this conversation as resolved Outdated

We should use %.1f Mib here for clearance.

We should use `%.1f Mib` here for clearance.
}
return nil
}