search

kepazon / mvc-mini-profiler

Automatically exported from code.google.com/p/mvc-mini-profiler
0 stars 0 forks source link

Modify XML help of AssembliesToExclude to make discovery of ExcludeAssembly() easier #148

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I found AssembliesToExclude and MethodsToExclude by browsing through the 
MiniProfiler.Settings namespace with intellisense, but could not figure out how 
to add to them (since they're both read-only IEnumerable) until I went to the 
MiniProfiler source code and found ExcludeMethod() and ExcludeAssembly(). 

It would improve usability if the XML help was changed:

            /// <summary>
            /// Assemblies to exclude from the stack trace report.
            /// Add to this using the <see cref="ExcludeAssembly"/> method.
            /// </summary>

Should apply to assemblies, methods, and types. 

This may sound trivial or stupid to any MiniProfiler dev's but consider you're 
used to the API. I am not, and with the current lack of documentation, 
intellisense is very important. My first reaction was "oh, that's dumb, it's 
IEnumerable but should be IList, I'm gonna fix it" .. not "oh, there must be 
somewhere else to configure this".

Original issue reported on code.google.com by gro...@gmail.com on 21 Mar 2012 at 7:44

GoogleCodeExporter commented 9 years ago

Original comment by sam.saff...@gmail.com on 3 Apr 2012 at 4:07