[forge-dev] Stacks - FORGE-2316
ggastald at redhat.com
Mon Jan 11 22:14:32 EST 2016
Yes, I did and I refactored the code removing the duplicated part. Thank
you for reviewing it. I think that adding a new annotation wouldn't help
much, therefore I added a new protected method in AbstractProjectCommand to
do the filtering (we can discuss this in the next meeting).
However there is one thing that hit my mind: by making the JavaEE7FacetImpl
a StackFacet, it doesn't matter if I have another stack, the JavaEE 7 stack
will always be activated when the JavaEE 7 dependencies are set in the
Perhaps I should store the stack information used in the project as part of
the project configuration (by using the ConfigurationFacet)?
Em 12/01/2016 00:42, "Lincoln Baxter, III" <lincolnbaxter at gmail.com>
> Did you see my comments on github?
> Lincoln Baxter, III
> "Simpler is better."
> On Mon, Jan 11, 2016 at 4:53 PM, George Gastaldi <ggastald at redhat.com>
>> Hello Forgers!
>> I would really appreciate your feedback on the Forge Stacks support. I
>> have provided a pull-request (https://github.com/forge/core/pull/592)
>> that I want to merge still this week with the changes.
>> I still need to document this feature, but it basically does what is
>> described in https://issues.jboss.org/browse/FORGE-2316
>> Regarding to the implementation details, I created a StacksFacet
>> interface (that's supposed to be implemented if you want to add a new
>> stack) that returns a Stack object (that is in charge of deciding if a
>> given ProjectFacet is supported). Stacks are built using the StackBuilder
>> I have added a --stack option in the project-new command that just calls
>> the StackFacet.install() method. In order to find the facet bound to the
>> project it is simply a matter of calling:
>> Let me know if you have any questions. I'd be more than happy to answer.
>> *George Gastaldi*
>> forge-dev mailing list
>> forge-dev at lists.jboss.org
> forge-dev mailing list
> forge-dev at lists.jboss.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the forge-dev