Output "The baseline file was updated." dialogue once about detect-secrets HOT 2 CLOSED

I'm pretty sure this is an artifact of the pre-commit engine, rather than our library. These are the reasons why I believe so:

1. We only output after all files are scanned

Source:

We only output this per invocation of the pre-commit hook, based on the list of filenames that are given to us. While we may be able to perform the logic to get the git diff --staged --name-only files, this would be essentially monkey-patching the pre-commit engine? It might work, seeing that we scan all files anyway, but it also seems like an anti-pattern.

2. Pre-commit performs xargs optimizations for parallel processing

Stack trace:

• https://github.com/pre-commit/pre-commit/blob/b8300268bf2d8797f28edf9063d7e5659ad4d535/pre_commit/commands/run.py#L123-L125
• https://github.com/pre-commit/pre-commit/blob/b8300268bf2d8797f28edf9063d7e5659ad4d535/pre_commit/repository.py#L100
• https://github.com/pre-commit/pre-commit/blob/809b7482df7b739014cb583c0793f495d9a949d0/pre_commit/languages/python.py#L148
• https://github.com/pre-commit/pre-commit/blob/9c6a1d80d6b94c86a1785a40a51389e83accac3e/pre_commit/languages/helpers.py#L90
• https://github.com/pre-commit/pre-commit/blob/9c6a1d80d6b94c86a1785a40a51389e83accac3e/pre_commit/xargs.py#L89

Essentially, this suggests that pre-commit does invoke our library multiple times, hence printing multiple statements.

I'll leave this issue open for discussions around whether performing the git diff ourselves is worth it; otherwise, my recommendation is to close this as "Won't Do" for the above reasons.

from detect-secrets.

We're going to close this issue as it hasn't received any update in a very long time. Feel free to re-open it if you think it's still relevant.

from detect-secrets.