Pipeline template: New pre-commit hooks (large files, merge conflicts)

CHANGELOG.md

@@ -13,6 +13,8 @@
 
 ### Template
 
+- New pre-commit hooks (large files, merge conflicts) in pipeline template ([#3935](https://github.com/nf-core/tools/pull/3935))
+
 ### Linting
 
 ### Modules

nf_core/pipeline-template/.hooks/block_pipeline_outdir.sh

@@ -0,0 +1,30 @@
+#!/usr/bin/env bash
+# This hook is used to block commits if they include staged files inside a directory
+# which also contains a subdirectory called `pipeline_info`. The purpose of this is to
+# prevent users from inadvertently committing output from pipeline test runs inside the
+# development directory.
+
+set -e
+
+# list staged files
+staged_files="$(git diff --cached --name-only)"
+
+status=0
+
+for file in $staged_files; do
+    file_dir=$(dirname "$file")
+
+    # Walk up the directory tree and check if the current directory contains a subdirectory called `pipeline_info`
+    # or the staged file is itself inside a directory called `pipeline_info`.
+    while [ "$file_dir" != "." ] && [ "$file_dir" != "/" ]; do
+        if [ $(basename "$file_dir") == "pipeline_info" ] || [ -d "$file_dir/pipeline_info" ]; then
+        echo "❌ Commit blocked: Please do not commit output from pipeline test runs to the pipeline code itself."
+        echo "Use 'git restore --staged <file>...' to remove the output files from the staging area before proceeding."
+        status=1
+        break
+        fi
+        file_dir=$(dirname "$file_dir")
+    done
+done
+
+exit "$status"

nf_core/pipeline-template/.pre-commit-config.yaml

@@ -25,3 +25,19 @@ repos:
               subworkflows/nf-core/.*|
               .*\.snap$
           )$
+      - id: check-added-large-files
+        args: [--maxkb=5000]
+        exclude: |
+          (?x)^(
+              .*ro-crate-metadata.json$|
+              .*\.snap$|
+              lib/nfcore_external_java_deps.jar$|
+              docs/.*\.(svg|pdf)$
+          )$
+      - id: check-merge-conflict
+  - repo: local
+    hooks:
+      - id: block-pipeline-outdir
+        name: Prevent committing output from pipeline test runs to the pipeline code itself
+        entry: ./.hooks/block_pipeline_outdir.sh
+        language: script

nf_core/pipelines/create/template_features.yml

@@ -438,6 +438,7 @@ code_quality:
         - ".prettierignore"
         - ".prettierrc.yml"
         - ".github/workflows/fix-linting.yml"
+        - ".hooks/block_pipeline_outdir.sh"
       short_description: "Use code linters"
       description: "The pipeline will include code linters and CI tests to lint your code: pre-commit, editor-config and prettier."
       help_text: |