Make repo-specific checkout methods private

Groovy doesn't actually enforce this restriction, but this helps make clear that
checkout_repo() is the intended entrypoint.

Signed-off-by: Bence Szépkúti <[email protected]>

Author: bensze01

vars/checkout_repo.groovy

@@ -86,7 +86,7 @@ String get_submodule_commit(String working_dir = '.', String submodule) {
     ).trim()
 }
 
-void checkout_framework_repo(BranchInfo info) {
+private void checkout_framework_repo(BranchInfo info) {
     if (env.TARGET_REPO == 'framework' && env.CHECKOUT_METHOD == 'scm') {
         checkout_report_errors(scm)
     } else {
@@ -104,7 +104,7 @@ void checkout_framework_repo(BranchInfo info) {
     }
 }
 
-void checkout_tf_psa_crypto_repo(BranchInfo info) {
+private void checkout_tf_psa_crypto_repo(BranchInfo info) {
     if (env.TARGET_REPO == 'tf-psa-crypto' && env.CHECKOUT_METHOD == 'scm') {
         checkout_report_errors(scm)
         if (!info.framework_override) {
@@ -140,7 +140,7 @@ void checkout_tf_psa_crypto_repo(BranchInfo info) {
     }
 }
 
-Map<String, String> checkout_tls_repo(BranchInfo info) {
+private Map<String, String> checkout_tls_repo(BranchInfo info) {
     if (info.repo != 'tls') {
         throw new IllegalArgumentException("checkout_tls_repo() called with BranchInfo for repo '$info.repo'")
     }

vars/gen_jobs.groovy

@@ -658,7 +658,7 @@ def gen_coverity_push_jobs(BranchInfo info) {
             try {
                 dir("src") {
                     deleteDir()
-                    checkout_repo.checkout_tls_repo(info)
+                    checkout_repo.checkout_repo(info)
                     sshagent([env.GIT_CREDENTIALS_ID]) {
                         analysis.record_inner_timestamps('container-host', job_name) {
                             // Git complains about non-fast-forward operations when trying to push a shallow commit