Skip to content

Note that Log implementors should call enabled#506

Merged
KodrAus merged 2 commits into
rust-lang:masterfrom
KodrAus:chore/enabled-requirement
Apr 28, 2022
Merged

Note that Log implementors should call enabled#506
KodrAus merged 2 commits into
rust-lang:masterfrom
KodrAus:chore/enabled-requirement

Conversation

@KodrAus
Copy link
Copy Markdown
Contributor

@KodrAus KodrAus commented Apr 20, 2022

Closes #472

This just notes that implementors of the Log trait should call their own enabled function somewhere in their log implementation to apply filtering.

Copy link
Copy Markdown
Collaborator

@Thomasdezeeuw Thomasdezeeuw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's also mentioned in the log method docs. But more docs don't hurt.

@KodrAus KodrAus merged commit b7e8147 into rust-lang:master Apr 28, 2022
EFanZh pushed a commit to EFanZh/log that referenced this pull request Jul 23, 2023
Bumps [miette](https://github.com/zkat/miette) from 5.3.0 to 5.4.1.
- [Release notes](https://github.com/zkat/miette/releases)
- [Changelog](https://github.com/zkat/miette/blob/main/CHANGELOG.md)
- [Commits](zkat/miette@miette-derive-v5.3.0...miette-derive-v5.4.1)

---
updated-dependencies:
- dependency-name: miette
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

Shouldn't log! check the equivalent of log_enabled! before calling into Log::log?

2 participants