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.

Sign up for GitHub

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

Jump to bottom

Support SemanticLogger logging with improve stdout logic and tagged logs #715

Merged

bensheldon merged 1 commit into main from semantic_logger

Sep 26, 2022

Owner

bensheldon commented Sep 26, 2022

Closes #667.

Semantic logger overrides ActiveSupport::Logger.logger_outputs_to? to always return true
Being extra defensive with the tagged logging construction; at expense of performance (but probably fine).


          Support SemanticLogger logging with improve stdout logic and tagged logs

71a91ea

bensheldon temporarily deployed to goodjob-semantic-logger-4kvgdq

September 26, 2022 20:30

Inactive

bensheldon mentioned this pull request

Clear up CLI logical expression to tee logs to stdout #714

Closed

bensheldon added the bug label

bensheldon merged commit 991a2d0 into main

bensheldon deleted the semantic_logger branch

September 26, 2022 21:02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug