Skip to content

Conversation

alexeyk99
Copy link

No description provided.

@filipw
Copy link
Owner

filipw commented May 14, 2016

thanks, can you please change tabs to spaces? there are a lot of changes showing up that shouldn't be treated as changes

/// <summary>
/// Gets or sets the list of HTTP verbs that needs to participate in cache.
/// </summary>
public string[] Verbs { get; private set; }
Copy link
Owner

Choose a reason for hiding this comment

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

can this be made HttpMethod[] (enum based), instead of string[]?

Copy link
Author

Choose a reason for hiding this comment

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

Unfortunately no. Only simple types may be used on attribute e.g [CacheOutputHttpVerbs(HttpMethod.Get)] - produces a compilation error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants