Skip to content

feat: add python implementation - #7

Merged
Niraj-Kamdar merged 25 commits into
mainfrom
nk/http-py
Apr 26, 2023
Merged

feat: add python implementation#7
Niraj-Kamdar merged 25 commits into
mainfrom
nk/http-py

Conversation

@dOrgJelli

@dOrgJellidOrgJelli commented Mar 2, 2023•
edited by Niraj-Kamdar
Loading

Copy link
Copy Markdown
Contributor

This PR adds http-plugin for the python client.

dOrgJelli
dOrgJelli commented Mar 6, 2023
View reviewed changes

@dOrgJellidOrgJelli left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Also @Niraj-Kamdar I don't think the __pycache__ directory & typings/ directory should be included in source control, as they're build artifacts (if I'm not mistaken).

Comment threadimplementations/py/polywrap-http-plugin/README.md Outdated
Comment threadimplementations/py/polywrap-http-plugin/polywrap_http_plugin/__init__.py Outdated
Comment threadimplementations/py/polywrap-http-plugin/polywrap_http_plugin/__init__.py Outdated
Comment threadimplementations/py/polywrap-http-plugin/tests/test_ipfs.py Outdated
Comment threadimplementations/py/polywrap-http-plugin/tests/test_plugin.py Outdated
Niraj-Kamdar
Niraj-Kamdar reviewed Apr 14, 2023
View reviewed changes
Comment threadimplementations/js/package.json
@Niraj-Kamdar

Niraj-Kamdar commented Apr 18, 2023

Copy link
Copy Markdown
Contributor

TODO: CI/CD

dOrgJelli
dOrgJelli commented Apr 20, 2023
View reviewed changes
Comment threadimplementations/py/README.md
Comment threadimplementations/py/schema.graphql Outdated
cbrzn
cbrzn reviewed Apr 24, 2023
View reviewed changes
Comment thread.github/workflows/http-plugin-py-cd.yaml Outdated
Comment threadimplementations/py/tests/integration/wrapper/wrap.wasm
@Niraj-Kamdar@cbrzn
Update .github/workflows/http-plugin-py-cd.yaml
76ecaaa
Co-authored-by: Cesar Brazon <cesarbrazon10@gmail.com>
@Niraj-Kamdar
Niraj-Kamdar merged commit 1de5638 into mainApr 26, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cbrzncbrzncbrzn left review comments
@krisbitneykrisbitneyAwaiting requested review from krisbitney
@namestynamestyAwaiting requested review from namesty
@nerfZaelnerfZaelAwaiting requested review from nerfZael
@pilekspileksAwaiting requested review from pileks
+1 more reviewer
@Niraj-KamdarNiraj-KamdarNiraj-Kamdar approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

http plugin implementation

3 participants

@dOrgJelli@Niraj-Kamdar@cbrzn