Skip to content

Conversation

LucioFranco
Copy link
Contributor

Description of changes:

  • Replace genawaiter for async-stream

By submitting this pull request

  • I confirm that my contribution is made under the terms of the Apache 2.0 license.
  • I confirm that I've made a best effort attempt to update all relevant documentation.

cc @davidbarsky

Copy link
Contributor

@davidbarsky davidbarsky left a comment

Choose a reason for hiding this comment

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

Thanks!

@davidbarsky davidbarsky merged commit f404617 into awslabs:master Jul 17, 2020
davidbarsky pushed a commit that referenced this pull request Jul 17, 2020
bahildebrand added a commit that referenced this pull request Feb 4, 2021
* reorg

* Replace `genawaiter` with `async-stream` (#240)

* don't depend on serde_derive directly

* Add Cargo.lock to gitignore; whoops.

* Cleanup docs; handle panics in lambda functions correctly.

* Create a reusable runtime struct + builder

* Log handler errors in runtime. Issue #241

* Added basic.rs example, comments to other examples

* Tidied up examples readme.

* Fixed headers in examples readme.

* Formatting fix in error-handling.rs.

* Fixed comment in lib.rs

* Corrected handler_fn interface misconception in basic.rs

Co-authored-by: Veetaha <[email protected]>

* Upper-cased and edited examples/readme.md

* Add log and tracing examples; fix `tracing` dependency features.

* disable time on `tracing_subscriber`

* Removed unused refs from basic.rs

* Added error handling for #241, interim, broken.

* Added err logging to lib.rs, consolidated examples

* Log panic as Debug+Display, improved examples

* client.rs formatting fix

* Fixed "Process exited" error in lib.rs.

* Added some tracing!() to lib.rs.

* Enforced Send+Sync for http Handler impl.

* Added docs and missing _vars to reduce warnings

* Removed unnecessary + Sync on handler's future

* All tests passing

* Remove deprecated simple_logger calls

* Remove Cargo.lock

* cargo fmt changes

Co-authored-by: David Barsky <[email protected]>
Co-authored-by: Lucio Franco <[email protected]>
Co-authored-by: Ubuntu <[email protected]>
Co-authored-by: rimutaka <[email protected]>
Co-authored-by: Veetaha <[email protected]>
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.

2 participants