Skip to content

Conversation

gewarren
Copy link
Contributor

@dotnet-bot dotnet-bot added this to the July 2020 milestone Jul 30, 2020
@gewarren gewarren requested review from BillWagner and safern July 30, 2020 14:16
@gewarren
Copy link
Contributor Author

gewarren commented Jul 30, 2020

The diff looks good to me. The property that was missing that alerted us to the missing assemblies is now present at https://review.docs.microsoft.com/en-us/dotnet/api/microsoft.extensions.logging.console.consoleloggeroptions.useutctimestamp?view=dotnet-plat-ext-5.0&branch=pr-en-us-4591.

@gewarren
Copy link
Contributor Author

@carlossanlop After we merge this, is there an easy way to find/bring in any missing doc comments?

Copy link
Contributor Author

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

Marked all new members.

</remarks>
</Docs>
</Member>
<Member MemberName="RedactLoggedHeaders">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New member.

</remarks>
</Docs>
</Member>
<Member MemberName="AddHttpClient&lt;TClient,TImplementation&gt;">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New member.

</remarks>
</Docs>
</Member>
<Member MemberName="AddHttpClient&lt;TClient,TImplementation&gt;">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New member.

<remarks>To be added.</remarks>
</Docs>
</Member>
<Member MemberName=".ctor">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New member.

<remarks>To be added.</remarks>
</Docs>
</Member>
<Member MemberName=".ctor">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New member.

<remarks>To be added.</remarks>
</Docs>
</Member>
<Member MemberName="ShouldRedactHeaderValue">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New member.

<remarks>To be added.</remarks>
</Docs>
</Member>
<Member MemberName="UseUtcTimestamp">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New member.

<remarks>To be added.</remarks>
</Docs>
</Member>
<Member MemberName="CreateInstance">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New member.

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

This LGTM @gewarren

Go ahead and :shipit:

@gewarren gewarren merged commit 9387360 into master Jul 31, 2020
@gewarren gewarren deleted the preview7-extra-extensions branch July 31, 2020 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants