[forge-dev] Stacks - FORGE-2316
Lincoln Baxter, III
lincolnbaxter at gmail.com
Tue Jan 12 17:04:35 EST 2016
Hmmmmm.... Is there a down side to that scenario? (The stack being
Lincoln Baxter, III
"Simpler is better."
On Mon, Jan 11, 2016 at 10:14 PM, George Gastaldi <ggastald at redhat.com>
> 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 pom.xml.
> 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
> forge-dev mailing list
> forge-dev at lists.jboss.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the forge-dev