Merge pull request #3394 from mirpedrol/fix-sync-gha

General: Fix sync GHA by removing quotes from parsed branch name
nf-core · Jan 15, 2025 · 7fa5d5b · 7fa5d5b
2 parents 6680f41 + 24e6c80
commit 7fa5d5b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/.github/workflows/sync.yml b/.github/workflows/sync.yml
@@ -97,7 +97,7 @@ jobs:
 
         run: |
           pushd nf-core/${{ matrix.pipeline }}
-          defaultBranch=$(grep -B5 -A5 "nextflowVersion" nextflow.config | grep "defaultBranch" | cut -d"=" -f2)
+          defaultBranch=$(grep -B5 -A5 "nextflowVersion" nextflow.config | grep "defaultBranch" | cut -d"=" -f2 | sed "s/'//g")
           if [ -z "$defaultBranch" ]; then
             defaultBranch="master"
           fi

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -21,6 +21,7 @@
 
 - Parameters schema validation: allow oneOf, anyOf and allOf with `required` ([#3386](https://github.com/nf-core/tools/pull/3386))
 - Run pre-comit when rendering template for pipelines sync ([#3371](https://github.com/nf-core/tools/pull/3371))
+- Fix sync GHA by removing quotes from parsed branch name ([#3394](https://github.com/nf-core/tools/pull/3394))
 - manifest.author is not required anymore ([#3397](https://github.com/nf-core/tools/pull/3397))
 
 ### Version updates