Status: New Owner: ---- CC: tudor.gi...@gmail.com, simon.de...@gmail.com Labels: Type-Defect Priority-Medium Component-Famix Milestone-4.3
New issue 520 by tudor.gi...@gmail.com: Integrate MooseChef http://code.google.com/p/moose-technology/issues/detail?id=520
MooseChef should get integrated into Famix.
Due to the traits, the only reasonable solution is to change directly the Famix definitions for now in a dedicated query method category.
When Pharo will provide a better solution to extend a class with a trait, we can revisit the implementation and decouple it into a separate package.
Updates: Cc: jannik.laval
Comment #1 on issue 520 by tudor.gi...@gmail.com: Integrate MooseChef http://code.google.com/p/moose-technology/issues/detail?id=520
Issue 438 has been merged into this issue.
Comment #2 on issue 520 by tudor.gi...@gmail.com: Integrate MooseChef http://code.google.com/p/moose-technology/issues/detail?id=520
Issue 260 has been merged into this issue.
Updates: Status: Started Labels: -Type-Defect Type-Enhancement
Comment #3 on issue 520 by simon.de...@gmail.com: Integrate MooseChef http://code.google.com/p/moose-technology/issues/detail?id=520
What I suggest for 4.3 is to integrate Chef core but keep Cook, as we know it is quite reliable. So people can start using Chef and bulletproof while we get rid of Cook.
For Chef, the next steps are: - finish testing a few things - rewrite Package Blueprints with Chef (and make it work with namespaces, see issue #439 ) [- replace/rewrite basic Cook queries ] --> this would be done in 4.4... [- also rewrite DSM]
Both Package Blueprints and DSM use Cook but also define their own queries when Cook was not enough. That's an interesting exercise, that's for sure!
Comment #4 on issue 520 by tudor.gi...@gmail.com: Integrate MooseChef http://code.google.com/p/moose-technology/issues/detail?id=520
I agree to integrate Chef and still keep the old extensions. In any case, the integration should happen directly in Famix-Core.
Comment #5 on issue 520 by tudor.gi...@gmail.com: Integrate MooseChef http://code.google.com/p/moose-technology/issues/detail?id=520
Do you take care of this?
Updates: Status: Fixed
Comment #6 on issue 520 by simon.de...@gmail.com: Integrate MooseChef http://code.google.com/p/moose-technology/issues/detail?id=520
(No comment was entered for this change.)
Comment #7 on issue 520 by tudor.gi...@gmail.com: Integrate MooseChef http://code.google.com/p/moose-technology/issues/detail?id=520
Thanks. There was only a small problem due to a reference to TAInstaller in the initialize of QueryResult. It should work now.
Comment #8 on issue 520 by simon.de...@gmail.com: Integrate MooseChef http://code.google.com/p/moose-technology/issues/detail?id=520
Ah yes, I always forget this one thing :)