Skip to content
This repository was archived by the owner on Dec 14, 2018. It is now read-only.

Adding documentation comments #416

Merged
merged 3 commits into from
Aug 8, 2018
Merged

Conversation

sebastienros
Copy link
Member

Adding missing class documentation

@Tratcher
Copy link
Member

Tratcher commented Aug 8, 2018

#414

@@ -5,6 +5,9 @@

namespace Microsoft.Extensions.Caching.Distributed
{
/// <summary>
/// Represents the cache options applied to an entry of the <see cref="IDistributedCache"/> instance.
Copy link
Contributor

Choose a reason for hiding this comment

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

Provides configuration for an entry in <see cref="IDistributedCache"/>

@@ -6,22 +6,70 @@

namespace Microsoft.Extensions.Caching.Distributed
{
/// <summary>
/// Represents a distributed cache of serialized values.
Copy link
Contributor

Choose a reason for hiding this comment

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

Contract for a distributed cache of serialized values.

Copy link
Member Author

Choose a reason for hiding this comment

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

Nope, consistency with IMemoryCache

@sebastienros sebastienros merged commit 5897f52 into release/2.2 Aug 8, 2018
@sebastienros sebastienros deleted the sebros/doc-comments branch August 8, 2018 22:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants