[MonoDevelop] Gendarme Addin (new version)

[Eli Yukelzon]

Lluis, thanks for the suggestion and bug report.
All the bugs that you mentioned were fixed, and the features implemented.

So here goes the third release:
http://gendarmeaddin.googlecode.com/files/gendarmeaddin-0.3.tar.gz

Changes:
* Several Bugfixes
* Result grouping options - now results can be ungrouped, grouped by target
project or by violation type
* Headers removed from result view

Regarding you point of bundling Gendarme with the plugin - you think
including the whole Gendarme distribution is a good idea? What if the user
wants to use the installed version and not the bundled one?


On 5/25/07, Lluis Sanchez <lluis at ximian.com> wrote:
>
> 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ximian.com/pipermail/monodevelop-list/attachments/20070526/0121aae6/attachment-0001.html