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.

Sign up for GitHub

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

Jump to bottom

Improve summaries for methods in AzureAppConfigurationOptions #467

Merged
merged 5 commits into from
Oct 5, 2023
Merged

Improve summaries for methods in AzureAppConfigurationOptions #467

Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
ea23e77
first draft clarify select summaries and fix usefeatureflags summary
amerjusupovic Sep 15, 2023
8b1e9a7
Merge branch 'preview' into ajusupovic/clarify-select-usage
amerjusupovic Sep 22, 2023
92e0843
Merge branch 'preview' into ajusupovic/clarify-select-usage
amerjusupovic Sep 29, 2023
ed90180
move default query message to class summary
amerjusupovic Sep 29, 2023
8d001a0
change mention of null label to no label
amerjusupovic Oct 5, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions src/Microsoft.Extensions.Configuration.AzureAppConfiguration/AzureAppConfigurationOptions.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@
namespace Microsoft.Extensions.Configuration.AzureAppConfiguration
{
/// <summary>
/// Options used to configure the behavior of an Azure App Configuration provider.
/// Options used to configure the behavior of an Azure App Configuration provider.
/// If neither <see cref="Select"/> nor <see cref="SelectSnapshot"/> is ever called, all key-values with null label are included in the configuration provider.
amerjusupovic marked this conversation as resolved.
Show resolved Hide resolved
/// </summary>
public class AzureAppConfigurationOptions
{
Expand Down Expand Up @@ -189,7 +190,7 @@ public AzureAppConfigurationOptions SelectSnapshot(string name)

/// <summary>
/// Configures options for Azure App Configuration feature flags that will be parsed and transformed into feature management configuration.
/// If no filtering is specified via the <cref="FeatureFlagOptions"> then all feature flags with no label are loaded.
/// If no filtering is specified via the <see cref="FeatureFlagOptions"/> then all feature flags with no label are loaded.
/// All loaded feature flags will be automatically registered for refresh on an individual flag level.
/// </summary>
/// <param name="configure">A callback used to configure feature flag options.</param>
Expand Down