Skip to content

API Debt - Microsoft.Extensions.Options#8514

Merged
gewarren merged 2 commits intodotnet:mainfrom
Nick-Stanton:Options-APIDebt
Oct 17, 2022
Merged

API Debt - Microsoft.Extensions.Options#8514
gewarren merged 2 commits intodotnet:mainfrom
Nick-Stanton:Options-APIDebt

Conversation

@Nick-Stanton
Copy link
Copy Markdown
Member

Summary

Following community contributions, there were only a couple constructor summaries missing from dotnet/runtime#43919.

Microsoft.Extensions.OptionsWrapper.Add and Remove were also missing, but they are obsolete as of dotnet/runtime@2402899.

Fixes dotnet/runtime#43919

@Nick-Stanton Nick-Stanton requested a review from a team as a code owner October 14, 2022 17:45
@ghost ghost assigned Nick-Stanton Oct 14, 2022
@Nick-Stanton Nick-Stanton requested a review from a team October 14, 2022 17:45
@ghost
Copy link
Copy Markdown

ghost commented Oct 14, 2022

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost
Copy link
Copy Markdown

ghost commented Oct 14, 2022

Tagging subscribers to this area: @dotnet/area-extensions-options
See info in area-owners.md if you want to be subscribed.

Issue Details

Summary

Following community contributions, there were only a couple constructor summaries missing from dotnet/runtime#43919.

Microsoft.Extensions.OptionsWrapper.Add and Remove were also missing, but they are obsolete as of dotnet/runtime@2402899.

Fixes dotnet/runtime#43919

Author: Nick-Stanton
Assignees: Nick-Stanton
Labels:

area-Extensions-Options

Milestone: -

@opbld30

This comment was marked as outdated.

@opbld32
Copy link
Copy Markdown

opbld32 commented Oct 14, 2022

Learn Build status updates of commit 6fa2c7b:

✅ Validation status: passed

File Status Preview URL Details
xml/Microsoft.Extensions.Options/OptionsCache`1.xml ✅Succeeded View
xml/Microsoft.Extensions.Options/ValidateOptionsResult.xml ✅Succeeded View

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:

@gewarren gewarren merged commit 95fcbd0 into dotnet:main Oct 17, 2022
@Nick-Stanton Nick-Stanton deleted the Options-APIDebt branch October 17, 2022 19:16
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.

API documentation debt - Microsoft.Extensions.Options

4 participants