Skip to content

Add option to output digest to stdout #264

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 19, 2025

Conversation

cwest
Copy link
Contributor

@cwest cwest commented Apr 27, 2025

This change introduces the ability for users to direct the output of the gitingest tool to standard output (stdout) instead of writing to a file. This is useful for piping the output to other commands or viewing it directly in the terminal.

Fixes #225

@cwest

This comment was marked as off-topic.

@drupol

This comment was marked as spam.

This change introduces the ability for users to
direct the output of the gitingest tool to
standard output (stdout) instead of writing to a
file. This is useful for piping the output to
other commands or viewing it directly in the
terminal.
Copilot

This comment was marked as outdated.

Click 8.2 removed the `mix_stderr` keyword from `CliRunner.__init__`.
The test suite now conditionally passes `mix_stderr=False` only when
the parameter is present (Click 8.0–8.1), keeping stdout/stderr
separation on older versions while remaining compatible with 8.2+.

No production code changes; succeeds on Python 3.8 with Click 8.1 and
Python 3.10+ with Click 8.2.
Copilot

This comment was marked as outdated.

Copilot

This comment was marked as outdated.

Copilot

This comment was marked as outdated.

@filipchristiansen filipchristiansen force-pushed the topic/stdout branch 2 times, most recently from 10944de to 5f39456 Compare June 19, 2025 06:39
@filipchristiansen filipchristiansen merged commit c656635 into coderamp-labs:main Jun 19, 2025
18 checks passed
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.

3 participants