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

appender(docs): add missing entry for Rotation::MINUTELY #1110

Merged

Conversation

salewski
Copy link
Contributor

The list introduced by:

"To use a Rotation, pick one of the following options:"

had en entry for all variations of Rotation except for Rotation::MINUTELY. This PR completes this list, with the doc test structured the same as the others in the list.

The newly added entry heading is "Minutely Rotation".

Side-note: I had to look up "minutely" in several dictionaries to convince myself that it is actually a legit English word that means "minute by minute" (in addition to the more familiar definition meaning "careful examination"), and was not just a cute symbol name in the code. It turns out that that usage of the word is legit! (But you folks already knew that) :-)

I performed both positive and negative tests to ensure that 'cargo test --doc' is correctly testing the change.

The list introduced by:

    "To use a Rotation, pick one of the following options:"

had en entry for all variations of Rotation except for
Rotation::MINUTELY. This changeset completes this list, with the doc
test structured the same as the others in the list.
@salewski salewski requested a review from a team as a code owner November 17, 2020 01:06
Copy link
Member

@davidbarsky davidbarsky left a comment

Choose a reason for hiding this comment

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

Whoops, thanks for fixing this!

@davidbarsky davidbarsky merged commit 783bdd1 into tokio-rs:master Nov 17, 2020
hawkw added a commit that referenced this pull request Dec 29, 2020
Changed

- **non_blocking**: Updated `crossbeam-channel` dependency to 0.5
  (#1031)

Fixed
- **non_blocking**: Fixed a race condition when logging on shutdown
  (#1125)
- Several documentation improvements (#1109, #1110, #941, #953)
hawkw added a commit that referenced this pull request Dec 29, 2020
Changed

- **non_blocking**: Updated `crossbeam-channel` dependency to 0.5
  (#1031)

Fixed
- **non_blocking**: Fixed a race condition when logging on shutdown
  (#1125)
- Several documentation improvements (#1109, #1110, #941, #953)
kaffarell pushed a commit to kaffarell/tracing that referenced this pull request May 22, 2024
Changed

- **non_blocking**: Updated `crossbeam-channel` dependency to 0.5
  (tokio-rs#1031)

Fixed
- **non_blocking**: Fixed a race condition when logging on shutdown
  (tokio-rs#1125)
- Several documentation improvements (tokio-rs#1109, tokio-rs#1110, tokio-rs#941, tokio-rs#953)
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.

2 participants