third-party/pre-commit
1
0
Fork 0
mirror of https://github.com/pre-commit/pre-commit.git synced 2026-04-15 18:11:48 +04:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Make the unstagged changes patch filename easier to understand

Browse source 
By having the basename of the repository directory and a
human readable time in the filename.

Instead of having something like this:
```
[WARNING] Unstaged files detected.
[INFO] Stashing unstaged files to /home/asottile/.cache/pre-commit/patch1591945458.
```

We'll now have something like this:
```
[WARNING] Unstaged files detected.
[INFO] Stashing unstaged files to /home/pierre/.cache/pre-commit/patch-pre-commit-2020-06-12-11-37-48.
```
This commit is contained in:
Pierre Sassoulas 2020-06-12 13:38:59 +02:00
parent 0e5eb19929
commit 330213e3d2
2 changed files with 6 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
pre_commit/git.py
View file

@ -42,6 +42,10 @@ def no_git_env(_env: Optional[EnvironT] = None) -> Dict[str, str]:
}
def get_basename() -> str:
return cmd_output('basename', get_root())[1].strip()
def get_root() -> str:
return cmd_output('git', 'rev-parse', '--show-toplevel')[1].strip()

3
pre_commit/staged_files_only.py
View file

@ -47,7 +47,8 @@ def _unstaged_changes_cleared(patch_dir: str) -> Generator[None, None, None]:
retcode=None,
)
if retcode and diff_stdout_binary.strip():
patch_filename = f'patch{int(time.time())}'
strftime = time.strftime('%Y-%m-%d-%H-%M-%S', time.gmtime())
patch_filename = f'patch-{git.get_basename()}-{strftime}'
patch_filename = os.path.join(patch_dir, patch_filename)
logger.warning('Unstaged files detected.')
logger.info(f'Stashing unstaged files to {patch_filename}.')