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

Update ServicesExtensions.cs #275

Merged
merged 2 commits into from
Feb 21, 2023
Merged

Conversation

pingvin1308
Copy link
Contributor

replace string format by index with names

it can be useful for log indexing systems that tries to index values that services passes into the string.

replace string format by index with names
@mgravell
Copy link
Member

mgravell commented Feb 9, 2023

Fair enough; found the docs: https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/?view=aspnetcore-7.0#log-message-template - checks out and makes sense

@mgravell
Copy link
Member

mgravell commented Feb 9, 2023

can you update /docs/releasenotes.md too, please? see existing content for format - this would be in the "unreleased" section currently

@mgravell
Copy link
Member

merging, thanks

@mgravell mgravell merged commit d8e445a into protobuf-net:main Feb 21, 2023
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.

3 participants