Skip to content

Add builder_from_env to use custom env var#13

Open
DaveWK wants to merge 2 commits into
softprops:mainfrom
DaveWK:CustomEnvVar
Open

Add builder_from_env to use custom env var#13
DaveWK wants to merge 2 commits into
softprops:mainfrom
DaveWK:CustomEnvVar

Conversation

@DaveWK

@DaveWK DaveWK commented Sep 3, 2022

Copy link
Copy Markdown

This adds a wrapper to Builder::from_env so a variable other than
RUST_LOG can be used

This also updates the log crate to the latest, where the feature has
been changed to "kv_unstable_std", as well as updates env_logger to 0.9

I added a WriteAdapter struct into the tests to capture log output into variables for testing

This adds a wrapper to Builder::from_env so a variable other than
RUST_LOG can be used

This also updates the log crate to the latest, where the feature has
been changed to "kv_unstable_std", as well as updates env_logger to 0.9
…mutex to avoid format changes impacting other tests
@DaveWK

DaveWK commented Sep 3, 2022

Copy link
Copy Markdown
Author

I have added some helpers to the tests to allow the logger to be replaced by a new instance. This also required making 2 of the tests run serially to avoid changing the filters/formatters that another test depended on.

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.

1 participant