-
Notifications
You must be signed in to change notification settings - Fork 171
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
Rust SDK #436
base: master
Are you sure you want to change the base?
Rust SDK #436
Conversation
Fix Licence Name in cargo.toml and Indentation in the twig templates
+ Allow package name to be changed in the sdk-generator for rust
…Onic/sdk-generator into rust-language-support
+ Added Example Generation + Changed example in README.md to use variable package name
Added first implementation of Exceptions into the Rust SDK
Co-authored-by: Damodar Lohani <[email protected]>
Co-authored-by: Damodar Lohani <[email protected]>
Add add_self_signed function to Client in SDK and Remove Cookies
+ Fixed using .unwrap on non-required objects causing a Panic - Fixed SDK Attempting to convert no content responses into JSON - Fixed a few display impl with objects - Fixed the way we deal with enum structs - Got attributes working correctly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only suggestion is working on reducing clone (which we have already talked about). Thanks for this!
Love to see it. I was thinking of creating a Rust SDK however I have no experience with it and it's a lot of work. Thank you for all that you have done |
Note: All Rust tests are passing in above commit, non-rust tests are failing. |
How is this going? Would love to see a Rust SDK for server-side ;) |
|
||
![License](https://img.shields.io/github/license/{{ sdk.gitUserName|url_encode }}/{{ sdk.gitRepoName|url_encode }}.svg?v=1) | ||
![Version](https://img.shields.io/badge/api%20version-{{ spec.version|url_encode }}-blue.svg?v=1) | ||
{% if sdk.warning %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would be nice if these if/endif blocks could be consistent with whitespace. I prefer:
{if}
thing..
{endif}```
without the whitespace between if & thing.
@PineappleIOnic any news on this? Can I help out in any way? |
This PR has Template files and Tests for an Appwrite Rust SDK,
I have also written the foobar tests and ran them myself to confirm that this SDK works correctly, This SDK Conforms as closely as it can with the Rust Language's Standards so it shouldn't compile with any warnings apart from unused imports which can't be avoided due to how the templates work.
Currently, the API is blocking I/O but I plan to add async support in the future behind a feature in the crate which I'm quite excited to do considering I'm not very good with async rust.
This PR also has full compatibility with Appwrite's response models making it as easy as possible for users to use the SDK. A Playground has already been written and tested and a PR is being prepared.