[MonoDevelop] Gendarme Addin

[Lluis Sanchez]

Hi Eli,

Good job on the add-in! It is a good start. Here are some
comments/ideas/bugs:

      * 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.

I hope it helps,
Lluis.


El dv 25 de 05 del 2007 a les 12:14 +0300, en/na Eli Yukelzon va
escriure:
> Ok. As per request the addin now uses the context sensitive menus,
> hope this more to your liking :)
> 
> The UI was minimalized as well.
> 
> The release is available at
> http://gendarmeaddin.googlecode.com/files/gendarmeaddin-0.2.tar.gz
> 
> Next release will make the run Async and add a status display.
> 
> On 5/25/07, Iñigo Illán Aranburu <kodeport at terra.es> wrote:
>         On vie, 2007-05-25 at 10:37 +0300, Eli Yukelzon wrote: 
>         > Inigo, thanks for the comments.
>         
>         No problem!
>         
>         >
>         > On 5/25/07, Iñigo Illán Aranburu <kodeport at terra.es> wrote:
>         >
>         >
>         >         I would suggest to put the "Run test" on some other
>         place. For 
>         >         example
>         >         on the Solution/Project right click menu. The button
>         wastes a
>         >         lot of
>         >         space IMHO. You could make the entry dynamic so you
>         can choose
>         >         what test 
>         >         to pass (I don't know if it's feasible). Also, I
>         don't get the
>         >         point of
>         >         having the violations tree discoverable.
>         >
>         > Regarding the conversion of Run button into a context Run
>         menu - I 
>         > think I'll take that suggestion.
>         > I am not clear on what you mean by 'discoverable' tree.
>         
>         For discoverable tree I mean the expander that hides the list
>         of
>         violations found.
>         
>         > 
>         >
>         >         Having the "Run gendarme tests" on the menu makes
>         the run on
>         >         all items
>         >         in the solution option useless, that is IMO a bit
>         confusing.
>         >
>         > If I move the Run command to the context menu, it will make
>         the option 
>         > useless, I will remove it.
>         >
>         > If you find any issues, feel free to drop me an email or add
>         a ticket
>         > at the project site.
>         
>         I think there should also be a visual clue of the state of the
>         running 
>         tests. It's a bit annoying not knowing if it has finished
>         pasing the
>         tests.
>         >
>         > Thanks
>         > _______________________________________________
>         > Monodevelop-list mailing list
>         > Monodevelop-list at lists.ximian.com
>         > http://lists.ximian.com/mailman/listinfo/monodevelop-list
>         
> 
> _______________________________________________
> Monodevelop-list mailing list
> Monodevelop-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/monodevelop-list