WINDUP-139  Create ModelModel and fill the graph.

I think I agree with that. I don't really see a reason for the prefix. It's
just verbose :)


On Thu, Jul 24, 2014 at 11:18 PM, Jesse Sightler <notifications@github.com>
wrote:

> In config/impl/src/main/java/org/jboss/windup/config/model/ModelModel.java:
>
> > +package org.jboss.windup.config.model;
> > +
> > +import com.tinkerpop.frames.Property;
> > +import com.tinkerpop.frames.modules.typedgraph.TypeValue;
> > +import org.jboss.windup.graph.model.WindupVertexFrame;
> > +
> > +/**
> > + * Model for Models - to have the information about models in the graph.
> > + * This serves e.g. for reporting.
> > + *
> > + * @author Ondrej Zizka, ozizka at redhat.com
> > + */
> > +@TypeValue("ModelModel")
> > +public interface ModelModel extends WindupVertexFrame
> > +{
> > +    public static final String CLASS = "class";
>
> Yeah, I haven't been doing the PROPERTY_ prefix thing so much now. It's
> not really accurate for a lot of the constants that are there (that
> actually represent edge labels instead of properties), and seems to be just
> adding verbosity.
>
> We should discuss it, but I think I am in favor of dropping all of the
> PROPERTY_ prefixes at this point.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/windup/windup/pull/147/files#r15384864>.
>



-- 
Lincoln Baxter, III
http://ocpsoft.org
"Simpler is better."