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

Add HN example, add try_ingest_stream and fix client construction, add tracing #12

Merged
merged 6 commits into from
Aug 8, 2022

Conversation

bahlo
Copy link
Member

@bahlo bahlo commented Jul 31, 2022

This PR adds a HN example and includes a few fixes and additions that were found while writing the example:

  • Client construction did not fall back to defaults if the environment variable was set, but empty
  • If you have a stream of Results the try_ingest_stream method is useful
  • Add support for tracing

@bahlo bahlo requested a review from schehata July 31, 2022 12:46
@bahlo bahlo force-pushed the arne/hn-example branch 6 times, most recently from 576d18a to f2527f7 Compare July 31, 2022 13:39
@bahlo bahlo changed the title Add HN example, add try_ingest_stream and fix client construction Add HN example, add try_ingest_stream and fix client construction, add tracing Jul 31, 2022
@bahlo bahlo force-pushed the arne/hn-example branch 2 times, most recently from 5ee0b3a to b1bbf0a Compare July 31, 2022 15:44
@bahlo bahlo marked this pull request as ready for review August 2, 2022 09:08
@bahlo bahlo force-pushed the arne/hn-example branch 4 times, most recently from 8805ae2 to 2736e59 Compare August 2, 2022 10:50
@bahlo bahlo mentioned this pull request Aug 6, 2022
If an env var was defined, but empty, env::var would return Some("").
This commit handles empty env vars the same as non-existent.
This method is useful if you have a stream of `Result`s.
It only works for personal tokens and the name is confusing.
@bahlo bahlo merged commit 9d396e2 into main Aug 8, 2022
@bahlo bahlo deleted the arne/hn-example branch August 8, 2022 11:26
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.

None yet

2 participants