fix: omit norun jobs when determining remote storage input file retrieval (#2854)

johanneskoester · web-flow · commit 37a7c7f9dbd7 · 2024-04-29T14:30:19.000+02:00
&lt;!--Add a description of your PR here--&gt;

### QC
&lt;!-- Make sure that you can tick the boxes below. --&gt;

* [x] The PR contains a test case for the changes or the changes are
already covered by an existing test case.
* [x] The documentation (`docs/`) is updated to reflect the changes or
this is not necessary (e.g. if the change does neither modify the
language nor the behavior or functionalities of Snakemake).
diff --git a/.github/workflows/codespell.yml b/.github/workflows/codespell.yml
@@ -21,3 +21,5 @@ jobs:
         uses: actions/checkout@v4
       - name: Codespell
         uses: codespell-project/actions-codespell@v2
+        with:
+          ignore_words_list: Crate,crate
diff --git a/snakemake/dag.py b/snakemake/dag.py
@@ -369,6 +369,7 @@ async def retrieve_storage_inputs(self, jobs=None, also_missing_internal=False):
                 (also_missing_internal and not shared_local_copies)
                 or self.is_external_input(f, job, not_needrun_is_external=True)
             )
+            and not job.is_norun
         }
 
         if to_retrieve:

Original file line number	Diff line number	Diff line change
`@@ -369,6 +369,7 @@ async def retrieve_storage_inputs(self, jobs=None, also_missing_internal=False):`
`369`	`369`	`(also_missing_internal and not shared_local_copies)`
`370`	`370`	`or self.is_external_input(f, job, not_needrun_is_external=True)`
`371`	`371`	`)`
	`372`	`+ and not job.is_norun`
`372`	`373`	`}`
`373`	`374`
`374`	`375`	`if to_retrieve:`