Skip to content

Conversation

JinsPeter
Copy link

The example in the documentation was using a wrong name for the attribute. Corrected the usage in the example

The example in the documentation was using a wrong name for the attribute. Corrected the usage in the example
Copy link
Contributor

@skomis-mm skomis-mm left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Member

@augustoproiete augustoproiete left a comment

Choose a reason for hiding this comment

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

LGTM

@JinsPeter
Copy link
Author

Thanks. So, Is there anything pending from my side to get this merged?

@augustoproiete
Copy link
Member

@JinsPeter No, you're good. Just giving folks time to 👀

@JinsPeter
Copy link
Author

JinsPeter commented Dec 6, 2021 via email

@nblumhardt nblumhardt merged commit 1aabe1d into serilog:dev Dec 8, 2021
@nblumhardt nblumhardt mentioned this pull request Apr 23, 2022
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.

5 participants