search

SamSaffron / MiniProfiler

A simple but effective mini-profiler for ASP.NET MVC, ASP.NET and Ruby.
http://miniprofiler.com
1.05k stars 221 forks source link

code cleanup #102

Closed jimmymain closed 11 years ago

jimmymain commented 11 years ago

Hi,

I have contributed before (rest support).

I would like to cleanup the code, and make it all stylecop compliant. Would you guys be happy to accept a pull request for all the necessary fixes and comments?

I don't want to do the work if you guys aren't keen to pull it. It will involve mostly cosmetic changes, but there will be quite a lot of them and it will perhaps make comparing prior versions a little trickier.

Cheers Craig.

SamSaffron commented 11 years ago

Personally I am fine with that, but for the .net portion is up to @dixon to make the call

We also talked about a folder restructure that was on the roadmap.

dixon commented 11 years ago

That sounds fine, but be mindful of the ruby version, since the UI portion is shared.

@SamSaffron heh, the folder structure didn't get any work done on my holiday; there were other items that needed work :(

jimmymain commented 11 years ago

o.k. I will try and have it done by Monday (I might need some time on the weekend). I will start with the .cs files and will keep an eye on the ruby code.