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

Document hostcalls::dispatch_http_call #201

Closed
wants to merge 2 commits into from

Conversation

younes-io
Copy link

I think functions should be documented to make it easy to find (regardless of the ABI, which is now being written, cf. proxy-wasm/spec#41)

@younes-io younes-io requested a review from PiotrSikora as a code owner June 18, 2023 12:12
@google-cla
Copy link

google-cla bot commented Jun 18, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@younes-io younes-io force-pushed the patch-1 branch 2 times, most recently from eb6770a to 60d9295 Compare June 18, 2023 12:44
@younes-io
Copy link
Author

is there any issue here ?

@PiotrSikora
Copy link
Member

is there any issue here ?

It doesn't pass tests (see Clippy failures).

Also, I believe that @mpwarres and @martijneken are currently working on the documentation for the whole SDK, so it might be worth waiting a bit for it for the sake of consistency. @mpwarres @martijneken do you have status update?

@mpwarres
Copy link

mpwarres commented Jul 6, 2023

Also, I believe that @mpwarres and @martijneken are currently working on the documentation for the whole SDK, so it might be worth waiting a bit for it for the sake of consistency. @mpwarres @martijneken do you have status update?

C++ SDK doc PR should be out end of this week, Rust SDK PR early next week.

@cuisongliu
Copy link

/// # Returns
///
/// A Result indicating the success or failure of the HTTP call. If the call is successful, the function returns an HTTP status code as a u32. If an error occurs, an Error object is returned.

There may be some problems with this description. It seems that the return is not an HTTP status code, but the code looks more like a token_id

@younes-io
Copy link
Author

@PiotrSikora : is this still relevant or shall I close it ?

@younes-io
Copy link
Author

@mpwarres : could you please tell if this PR is still relevant or is obsolete (due to the latest work on docs, etc) ?

@younes-io
Copy link
Author

@mpwarres @martijneken @PiotrSikora: For your information, I'm closing this as it's been more than one year and a half, and it seems abandoned.

@younes-io younes-io closed this Oct 19, 2024
@antonengelhardt
Copy link

@younes-io Maybe you want to take a look at #226? I tried to document some functions and added CI tests to check if the examples compile.

@younes-io
Copy link
Author

@younes-io Maybe you want to take a look at #226? I tried to document some functions and added CI tests to check if the examples compile.

Will do! Thank you!

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.

5 participants