Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create CIP #9

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

pontusmelke
Copy link

Something went wrong with the old PR when moving repo. Please kill the old one when all relevant comments have been moved over.

@pontusmelke pontusmelke changed the title Create cip Create CIP Oct 26, 2015
@systay systay mentioned this pull request Oct 27, 2015

== Proposal

`CREATE` is used to create new nodes and relationships into the graph, as well as set initial labels, type, and properties on the new entities.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"create new nodes and relationships into the graph" -> "add new nodes and relationships to the graph"

@nawroth
Copy link

nawroth commented Oct 28, 2015

in-rel = "<-[", [identifier], type, [property], "]-" ;
label = ":", identifier ;
type = label ;
property = "{", param | {identifier, ":", literal | param}, "}" ;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the param production already contains the "{" and "}", and should this be on the "top level" of the property production.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not understand what you mean.

@petraselmer
Copy link

The title needs to be changed: "CIP2015-10-12 - CREATE" -> "CIP2015-10-12 CREATE"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants