Skip to content

Conversation

gewarren
Copy link
Contributor

@gewarren gewarren commented Jun 1, 2022

Build.Reason:Manual by Genevieve Warren
Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=296273&view=results
source_repo.branch:net7-pre4
source_repo.url:https://apidrop.visualstudio.com/_git/binaries

 Build.Reason:Manual by Genevieve Warren
 Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=296273&view=results
 source_repo.branch:net7-pre4
 source_repo.url:https://apidrop.visualstudio.com/_git/binaries
@ghost ghost assigned gewarren Jun 1, 2022
@ghost ghost added the area-Meta Concerns something that extends across runtime area boundaries, for example, IDisposable. label Jun 1, 2022
@gewarren
Copy link
Contributor Author

gewarren commented Jun 1, 2022

cc @carlossanlop

@opbld31

This comment was marked as outdated.

 Build.Reason:Manual by Genevieve Warren
 Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=296367&view=results
 source_repo.branch:net7-pre4
 source_repo.url:https://apidrop.visualstudio.com/_git/binaries
@gewarren gewarren requested a review from kouvel as a code owner June 2, 2022 06:31
@opbld34

This comment was marked as outdated.

@opbld32

This comment was marked as outdated.

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.

For the renamed types, there's some data loss in the documentation comments that we can address in a separate PR.

<typeparam name="TSelf">The type that implements this interface.</typeparam>
<typeparam name="TResult">The type that contains the additive identify of <typeparamref name="TSelf" />.</typeparam>
<summary>Defines a mechanism for getting the additive identity of a given type.</summary>
<typeparam name="TSelf">To be added.</typeparam>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment loss here.

Copy link
Contributor

Choose a reason for hiding this comment

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

I can take care of this with the porting tool.

</ReturnValue>
<Docs>
<summary>Gets the additive identity of the current type.</summary>
<summary>To be added.</summary>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

And here.

<typeparam name="TOther">The type that will is used in the operation with <typeparamref name="TSelf" />.</typeparam>
<typeparam name="TResult">The type that contains the result of <typeparamref name="TSelf" /> op <typeparamref name="TOther" />.</typeparam>
<summary>Defines a mechanism for performing bitwise operations over two values.</summary>
<typeparam name="TSelf">To be added.</typeparam>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment loss. Looks like we'll have to manually port the doc comments for the renamed interfaces.

<Docs>
<typeparam name="TSelf">The type that implements this interface.</typeparam>
<summary>Defines a mechanism for parsing a string to a value.</summary>
<typeparam name="TSelf">To be added.</typeparam>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment loss.

<Docs>
<typeparam name="TSelf">The type that implements this interface.</typeparam>
<summary>Defines a mechanism for parsing a span of characters to a value.</summary>
<typeparam name="TSelf">To be added.</typeparam>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment loss.

@opbld30
Copy link

opbld30 commented Jun 2, 2022

Docs Build status updates of commit 6986b69:

✅ Validation status: passed

File Status Preview URL Details
xml/FrameworksIndex/dotnet-plat-ext-7.0.xml ✅Succeeded
xml/FrameworksIndex/net-7.0.xml ✅Succeeded
xml/Microsoft.Extensions.Caching.Memory/IMemoryCache.xml ✅Succeeded View
xml/Microsoft.Extensions.Caching.Memory/MemoryCache.xml ✅Succeeded View
xml/Microsoft.Extensions.Caching.Memory/MemoryCacheOptions.xml ✅Succeeded View
xml/Microsoft.Extensions.Caching.Memory/MemoryCacheStatistics.xml ✅Succeeded View
xml/Microsoft.Extensions.Configuration.Xml/XmlDocumentDecryptor.xml ✅Succeeded View
xml/Microsoft.Extensions.Configuration.Xml/XmlStreamConfigurationProvider.xml ✅Succeeded View
xml/Microsoft.Extensions.Configuration/ChainedConfigurationProvider.xml ✅Succeeded View
xml/Microsoft.Extensions.Configuration/XmlConfigurationExtensions.xml ✅Succeeded View
xml/Microsoft.Extensions.DependencyInjection/ServiceCollection.xml ✅Succeeded View
xml/Microsoft.Extensions.Hosting.Internal/HostingEnvironment.xml ✅Succeeded View
xml/Microsoft.Extensions.Hosting/Host.xml ✅Succeeded View
xml/Microsoft.Extensions.Hosting/HostingHostBuilderExtensions.xml ✅Succeeded View
xml/Microsoft.Extensions.Http/HttpMessageHandlerBuilder.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Abstractions/LogEntry`1.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/ConsoleFormatter.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/ConsoleFormatterOptions.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/ConsoleLoggerOptions.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.Console/ConsoleLoggerProvider.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.EventLog/EventLogLoggerProvider.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.EventLog/EventLogSettings.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging.TraceSource/TraceSourceLoggerProvider.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging/ILogger.xml ✅Succeeded View
xml/Microsoft.Extensions.Logging/LoggerExtensions.xml ✅Succeeded View

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Copy link
Contributor

@carlossanlop carlossanlop left a comment

Choose a reason for hiding this comment

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

LGTM @gewarren . I'll make sure the porting tool addresses the renamed API documentation.

@gewarren gewarren merged commit 792c515 into main Jun 2, 2022
@gewarren gewarren deleted the net7-preview4 branch June 2, 2022 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Meta Concerns something that extends across runtime area boundaries, for example, IDisposable.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants