[MonoDevelop] Gendarme Addin
lluis at ximian.com
Fri May 25 05:59:43 EDT 2007
Good job on the add-in! It is a good start. Here are some
* You can minimalize the UI even more by hiding the header of the
* 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,
El dv 25 de 05 del 2007 a les 12:14 +0300, en/na Eli Yukelzon va
> 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
> 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
> 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
> > 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
> tests. It's a bit annoying not knowing if it has finished
> pasing the
> > 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
More information about the Monodevelop-list