[forge-dev] Add vs New

[Antonio Goncalves]

Yes. I've created a JIRA for Forge 3.x :

https://issues.jboss.org/browse/FORGE-2277

Antonio

2015-03-13 23:10 GMT+01:00 George Gastaldi <ggastald at redhat.com>:

> That makes sense, however renaming these commands will break existing
> scripts. This should be something to be considered for Forge 3.x
>
>
>
> Em 13/03/2015, às 19:00, Antonio Goncalves <antonio.mailing at gmail.com>
> escreveu:
>
> Hi all,
>
> I'm a bit particular on wording because I think that the right word makes
> things easier for the new comer. I'm implementing a new UI command to add
> an injection point to a class. So, the name of the command would be
> cdi-add-injection-point. But then I started to have a look at the other
> xxx-*add*-yyy commands :
>
> addon-add-dependency
> project-add-dependencies
> project-add-managed-dependencies
> project-add-repository
> java-add-annotation
> constraint-add
>
> They all add something, into something already existing. If we take this
> definition for granted, shouldn't the following commands be renamed *add*
> instead of *new* :
>
> jpa-new-named-query
> cdi-new-conversation
> java-new-enum-const
> java-new-field
> java-new-method
> jpa-new-field
>
>
>
> --
> Antonio Goncalves
> Software architect and Java Champion
>
> Web site <http://www.antoniogoncalves.org/> | Twitter
> <http://twitter.com/agoncal> | LinkedIn
> <http://www.linkedin.com/in/agoncal> | Paris JUG
> <http://www.parisjug.org/> | Devoxx France <http://www.devoxx.fr/>
>
> _______________________________________________
> forge-dev mailing list
> forge-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/forge-dev
>
>
> _______________________________________________
> forge-dev mailing list
> forge-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/forge-dev
>



-- 
Antonio Goncalves
Software architect and Java Champion

Web site <http://www.antoniogoncalves.org/> | Twitter
<http://twitter.com/agoncal> | LinkedIn <http://www.linkedin.com/in/agoncal>
 | Paris JUG <http://www.parisjug.org/> | Devoxx France
<http://www.devoxx.fr/>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/forge-dev/attachments/20150314/e6a9f35c/attachment.html