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

Generate Python SDK #156

Merged
merged 36 commits into from
Jun 1, 2018
Merged

Generate Python SDK #156

merged 36 commits into from
Jun 1, 2018

Conversation

jenow
Copy link
Contributor

@jenow jenow commented May 30, 2018

What does this PR do ?

Generate python SDK

How should this be manually tested?

Make the sdk

cd internal/wrappers
make python

Execute test

cd internal/wrappers/build/python
LD_PRELOAD=./ python3 test.py

Depends on #155

@jenow jenow self-assigned this May 30, 2018
@jenow jenow added the blocked label May 30, 2018
@codecov-io
Copy link

codecov-io commented May 30, 2018

Codecov Report

Merging #156 into 1.x will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##              1.x     #156   +/-   ##
=======================================
  Coverage   89.01%   89.01%           
=======================================
  Files         244      244           
  Lines        4406     4406           
=======================================
  Hits         3922     3922           
  Misses        452      452           
  Partials       32       32
Impacted Files Coverage Δ
kuzzle/query.go 69.64% <100%> (ø) ⬆️
realtime/publish.go 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a164cee...460c710. Read the comment docs.

@etrousset etrousset merged commit 9d93a1f into 1.x Jun 1, 2018
@jenow jenow deleted the swig-python branch June 4, 2018 07:45
@jenow jenow mentioned this pull request Sep 10, 2018
@jenow jenow mentioned this pull request Jun 5, 2019
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.

5 participants