use VALUE_ERROR as exit code for wrong download_repo

lexming · boegel · web-flow · commit b5d5bc8c0b79 · 2024-09-17T14:00:10.000+02:00
Co-authored-by: Kenneth Hoste &lt;kenneth.hoste@ugent.be&gt;
diff --git a/easybuild/tools/github.py b/easybuild/tools/github.py
@@ -404,7 +404,7 @@ def download_repo(repo=GITHUB_EASYCONFIGS_REPO, branch=None, commit=None, accoun
         latest_commit_sha = fetch_latest_commit_sha(repo, account, branch, github_user=github_user)
     else:
         raise EasyBuildError(
-            "Either branch or commit should be specified in download_repo", exit_code=EasyBuildExit.OPTION_ERROR
+            "Either branch or commit should be specified in download_repo", exit_code=EasyBuildExit.VALUE_ERROR
         )
 
     expected_path = os.path.join(path, extracted_dir_name)

Original file line number	Diff line number	Diff line change
`@@ -404,7 +404,7 @@ def download_repo(repo=GITHUB_EASYCONFIGS_REPO, branch=None, commit=None, accoun`
`404`	`404`	`latest_commit_sha = fetch_latest_commit_sha(repo, account, branch, github_user=github_user)`
`405`	`405`	`else:`
`406`	`406`	`raise EasyBuildError(`
`407`		`- "Either branch or commit should be specified in download_repo", exit_code=EasyBuildExit.OPTION_ERROR`
	`407`	`+ "Either branch or commit should be specified in download_repo", exit_code=EasyBuildExit.VALUE_ERROR`
`408`	`408`	`)`
`409`	`409`
`410`	`410`	`expected_path = os.path.join(path, extracted_dir_name)`