Skip to content
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

Fix stdout/err log errors introduced by PR #3347 #3379

Merged
merged 1 commit into from
Apr 22, 2024

Commits on Apr 20, 2024

  1. Fix stdout/err log errors introduced by PR #3347

    PR #3347 attempted to do case-based logging of all the different kinds
    of stdout/err specification.
    
    It failed to capture some of the cases involving os.PathLike, and so
    after PR #3347, those cases would log an ERROR that the specification
    was unknown. This new behavior is only a new ERROR logging message -
    PR #3347 did not change other behaviour.
    
    This PR also amends a rich test of stdout/err specification types
    introduced in PR #3363 to check that no ERROR messages are logged during
    these tests.
    benclifford committed Apr 20, 2024
    Configuration menu
    Copy the full SHA
    7618686 View commit details
    Browse the repository at this point in the history