Skip to content

docs(skill): fix stdout/stderr guidance in usethis-prek-hook-bespoke-create#1848

Merged
nathanjmcdougall merged 2 commits into
mainfrom
copilot/fix-precommit-hooks-output
Apr 7, 2026
Merged

docs(skill): fix stdout/stderr guidance in usethis-prek-hook-bespoke-create#1848
nathanjmcdougall merged 2 commits into
mainfrom
copilot/fix-precommit-hooks-output

Conversation

Copilot AI commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

The usethis-prek-hook-bespoke-create skill gave incorrect guidance: print errors to stderr and a success summary to stdout. Pre-commit captures stdout as its primary display stream — errors on stderr can be silently dropped, and a success message on stdout creates a false "always passes" appearance.

Changes

  • usethis-prek-hook-bespoke-create/SKILL.md (v1.3 → v1.4)
    • Replace "print errors to stderr / success to stdout" with: print all output to stdout
    • Add explicit guidance: no success messages — the framework already shows Passed/Failed

…dance

Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/cc822449-8f7d-40ef-8f1e-9f5ddeb9d39c

Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix pre-commit hooks to print all output to stdout docs(skill): fix stdout/stderr guidance in usethis-prek-hook-bespoke-create Apr 7, 2026
Copilot AI requested a review from nathanjmcdougall April 7, 2026 07:14
@nathanjmcdougall nathanjmcdougall marked this pull request as ready for review April 7, 2026 08:19
@nathanjmcdougall nathanjmcdougall merged commit e7b63b6 into main Apr 7, 2026
1 check passed
@nathanjmcdougall nathanjmcdougall deleted the copilot/fix-precommit-hooks-output branch April 7, 2026 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

agent: pre-commit hooks must print all output to stdout, not stderr

2 participants