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

Fix RVT, RET types to Strings #51

Merged
merged 1 commit into from
Mar 20, 2016
Merged

Conversation

laughedelic
Copy link
Member

To Strings, because in any DB any kind of keys/labels have String representation and it's enough for all manipulations with graph element types in angulillos.

@eparejatobes right?

@laughedelic laughedelic self-assigned this Mar 18, 2016
@eparejatobes
Copy link
Member

Yup

@laughedelic laughedelic changed the title Consider fixing RVT, RET types Fix RVT, RET types to Strings Mar 19, 2016
@laughedelic laughedelic changed the title Fix RVT, RET types to Strings Consider fixing RVT, RET types Mar 19, 2016
@laughedelic laughedelic changed the title Consider fixing RVT, RET types Fix RVT, RET types to Strings Mar 19, 2016
@eparejatobes
Copy link
Member

Good! 👍 LGTM

@laughedelic laughedelic merged commit 834d3de into master Mar 20, 2016
@laughedelic laughedelic deleted the refactoring/fix-rvt-ret branch March 20, 2016 00:53
laughedelic added a commit that referenced this pull request May 10, 2016
* A lot of general code refactoring (most of it in #62):
    - Setters return the subject for chaining calls
    - Added arity-specific methods in/out-E/V methods with default implementations
    - Moved all the methods from TypedGraph to the elements interfaces
    - Split Arity on two: From/To; Added FromArity to Property
    - Added Unique and NonUnique Property types to fix the arity
    - Added vertex/edge types list to the typed graph and properties list to each typed element
    - Added UntypedGraphSchema interface for automatic schema creation
* See also #46, #51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants