Apply suggestions from code review

Co-authored-by: Gabriel Nützi <[email protected]>
gabyx · Oct 3, 2024 · fe148cb · fe148cb
1 parent 5da6231
commit fe148cb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/githooks/apps/runner/settings.go b/githooks/apps/runner/settings.go
@@ -21,7 +21,7 @@ type HookSettings struct {
 	HookName      string // Name of the hook.
 	HookDir       string // Directory of the hook.
 	HookNamespace string // Namespace of this repositorie's Githooks.
-	ChecksumsDir  string // Name of the directory with the trust checksums
+	ChecksumsDir  string // Name of the directory with the trust checksums.
 
 	IsRepoTrusted              bool               // If the repository is a trusted repository.
 	SkipNonExistingSharedHooks bool               // If Githooks should skip non-existing shared hooks.

diff --git a/githooks/hooks/trusted.go b/githooks/hooks/trusted.go
@@ -248,7 +248,7 @@ func GetChecksumDirectoryGitDir(gitDir string) string {
 
 	exists, _ := cm.IsPathExisting(path.Join(gitDir, conf))
 	switch {
-	case !strs.IsEmpty(conf) && exists:
+	case !strs.IsEmpty(conf):
 		return path.Join(gitDir, conf)
 	default:
 		return path.Join(gitDir, ".githooks.checksums")