mookme icon indicating copy to clipboard operation
mookme copied to clipboard

Published 20 hours ago • verified publisher icon Escape-Technologies

The file arguments account for deleted files as well

Open LMaxence opened this issue 2 years ago • 3 comments

When using the argument {matchedFiles} for instance in a hook, it will list changed file of the package as expected, but it will also list deleted files, leading to errors sometimes.

Here, the file `` was deleted which breaks my pre-commit hook and made me do a commit --no-verify.

image

LMaxence avatar May 11 '22 08:05 LMaxence

I happened across this bug as well with renamed python files in mookme version 2.3.0.

jeffvswanson avatar Jun 14 '23 16:06 jeffvswanson

I opened a very primitive PR to fix this:

https://github.com/Escape-Technologies/mookme/pull/132

jfb-workhelix avatar Jun 24 '24 21:06 jfb-workhelix