search

reflog / gendarmeaddin

Automatically exported from code.google.com/p/gendarmeaddin
0 stars 0 forks source link

Lluis' comments #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
     * You can minimalize the UI even more by hiding the header of the
       treview.
     * If there is more than one assembly in the solution, it might be
       useful to see the errors grouped by assembly. In fact there are
       different ways the info can be grouped, for example, it might
       also be useful to group by rule type. Maybe it could offer
       several grouping options? just an idea.
     * It makes MD crash if the rules file can't be found.
     * It also crashes if one of the assemblies can't be found. It may
       happen if the project has not yet been compiled.
     * It crashed once in MonoDevelop.GendarmeDisplay.getSource with a
       null ref exception, when trying to show the context menu.
     * You don't need to bundle Mono.Cecil in the add-in, nor reference
       it from the .addin file, since it is already included by MD.
     * I think that the add-in should be self-contained. Right now it
       depends on the gendarme rules to be available somewhere, but I
       think it would be more practical to just bundle the rules with
       the add-in. If Gendarme is changed to use Mono.Addins someday,
       everything will be more simple, since rules can just be add-ins.

Original issue reported on code.google.com by ref...@gmail.com on 26 May 2007 at 11:40

GoogleCodeExporter commented 9 years ago 
* You can minimalize the UI even more by hiding the header of the
       treview.

Done

     * If there is more than one assembly in the solution, it might be
       useful to see the errors grouped by assembly. In fact there are
       different ways the info can be grouped, for example, it might
       also be useful to group by rule type. Maybe it could offer
       several grouping options? just an idea.

Done
     * It makes MD crash if the rules file can't be found.
Done
     * It also crashes if one of the assemblies can't be found. It may
       happen if the project has not yet been compiled.
Done
     * It crashed once in MonoDevelop.GendarmeDisplay.getSource with a
       null ref exception, when trying to show the context menu.
Done
     * You don't need to bundle Mono.Cecil in the add-in, nor reference
       it from the .addin file, since it is already included by MD.
Done
     * I think that the add-in should be self-contained. Right now it
       depends on the gendarme rules to be available somewhere, but I
       think it would be more practical to just bundle the rules with
       the add-in. If Gendarme is changed to use Mono.Addins someday,
       everything will be more simple, since rules can just be add-ins.
Open for discussion :)

Original comment by ref...@gmail.com on 26 May 2007 at 12:51