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.

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

ktesting + textlogger config api #368

Merged
merged 2 commits into from
Feb 9, 2023
Merged

ktesting + textlogger config api #368

merged 2 commits into from
Feb 9, 2023

Commits on Feb 9, 2023

  1. ktesting: support verbosity changes at runtime 

    Being able to change the verbosity at runtime is useful. It is already
supported by the underlying code, ktesting and its Config struct just didn't
expose it.
    @pohly
    pohly committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    e092d89 View commit details
    Browse the repository at this point in the history

  2. textlogger: verbosity changes through public API 

    By embedding *verbosity.VState in Config, users of the package
already had access to V and VModule, but that had two drawbacks:
- not easy to discover
- unclean separate between internal and external API

Providing explicit functions is better.
    @pohly
    pohly committed Feb 9, 2023
    Configuration menu
    Copy the full SHA
    a0fea0c View commit details
    Browse the repository at this point in the history