Skip to content

Bringing dgraph client up to speed with API version 0.9.4+ #19

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

Closed
wants to merge 22 commits into from

Conversation

kochhar
Copy link
Contributor

@kochhar kochhar commented Jan 9, 2018

This isn't completely ready for merge as yet. A few integration tests need to be completed for it to be up and running. I thought it would be good to start getting more eyes on this.

@kochhar
Copy link
Contributor Author

kochhar commented Jan 20, 2018

Integration tests requested are now added. Please take a look at this PR.

You can run tests via

$ python tests/test_acct_upsert.py
$ python tests/test_bank.py
$ python tests/test_txn.py

@pawanrawal pawanrawal self-assigned this Jan 31, 2018
@aztennenbaum
Copy link

Hey, I was just curious what still needs to be done in order to get this pull request merged.

I'm hoping to use dgraph python in a cubesat mission that's going to low earth orbit

@manishrjain
Copy link
Contributor

This repo wasn't linked to the ICLA. Can you please raise this PR again, so that the bot triggers and you can sign it? We need that in place to accept PRs.

@aztennenbaum
Copy link

@manishrjain Ah oops, I see you were talking to @kochhar

@kochhar
Copy link
Contributor Author

kochhar commented Feb 1, 2018

@manishrjain sure. will close this PR and re-open it.

@kochhar kochhar closed this Feb 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants