René's URL Explorer Experiment

Title: Fix hooks printing errors to stderr instead of stdout by Copilot · Pull Request #1844 · usethis-python/usethis-python · GitHub

Open Graph Title: Fix hooks printing errors to stderr instead of stdout by Copilot · Pull Request #1844 · usethis-python/usethis-python

X Title: Fix hooks printing errors to stderr instead of stdout by Copilot · Pull Request #1844 · usethis-python/usethis-python

Description: Pre-commit/prek hooks were printing violation output to stderr and a success message to stdout. Since prek captures stdout as the primary display stream, errors could go unseen while "No forbidden patterns found in docstrings." was always the visible output. Changes hooks/check-docstring-substrings.py, check-comment-keywords.py, check-banned-words.py: Print violations to stdout, remove redundant success messages (the framework already shows Passed/Failed), drop unused sys imports Before/after # Before: errors invisible, success message always shown if violations: print("ERROR: ...", file=sys.stderr) # potentially unseen return 1 print("No forbidden patterns found in docstrings.") # always visible # After: errors visible, no noise on success if violations: print("ERROR: ...") # stdout, captured by prek return 1 return 0 Lesson Filed #1845 — pre-commit hooks must use stdout for all output (general CLI convention of errors→stderr conflicts with the pre-commit captured-output model).

Open Graph Description: Pre-commit/prek hooks were printing violation output to stderr and a success message to stdout. Since prek captures stdout as the primary display stream, errors could go unseen while "No forbi...

X Description: Pre-commit/prek hooks were printing violation output to stderr and a success message to stdout. Since prek captures stdout as the primary display stream, errors could go unseen while "No f...

Opengraph URL: https://github.com/usethis-python/usethis-python/pull/1844

X: @github

direct link

Domain: github.com

Links:

Viewport: width=device-width