move patch discarding inside try for staged_files_only

there's a rare race outlined in pre-commit#2287
saravankrish · Jul 7, 2023 · 138f726 · 138f726
1 parent a171f40
commit 138f726
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pre_commit/staged_files_only.py b/pre_commit/staged_files_only.py
@@ -66,9 +66,9 @@ def _unstaged_changes_cleared(patch_dir: str) -> Generator[None, None, None]:
 
         # prevent recursive post-checkout hooks (#1418)
         no_checkout_env = dict(os.environ, _PRE_COMMIT_SKIP_POST_CHECKOUT='1')
-        cmd_output_b(*_CHECKOUT_CMD, env=no_checkout_env)
 
         try:
+            cmd_output_b(*_CHECKOUT_CMD, env=no_checkout_env)
             yield
         finally:
             # Try to apply the patch we saved