Skip to content

docs: clarify timeout option #315

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Apr 28, 2022
Merged

docs: clarify timeout option #315

merged 2 commits into from
Apr 28, 2022

Conversation

bednar
Copy link
Contributor

@bednar bednar commented Apr 27, 2022

Related to #88 (comment)

Proposed Changes

Clarified the meaning of timeout.

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • dotnet test completes successfully
  • Commit messages are in semantic format
  • Sign CLA (if not already signed)

@codecov-commenter
Copy link

codecov-commenter commented Apr 27, 2022

Codecov Report

Merging #315 (2e6f375) into master (cccc410) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #315   +/-   ##
=======================================
  Coverage   88.13%   88.13%           
=======================================
  Files          76       76           
  Lines        6526     6526           
=======================================
  Hits         5752     5752           
  Misses        774      774           
Impacted Files Coverage Δ
Client/Configurations/Influx2.cs 67.85% <ø> (ø)
Client/InfluxDBClientFactory.cs 88.40% <ø> (ø)
Client/InfluxDBClientOptions.cs 95.26% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cccc410...2e6f375. Read the comment docs.

@bednar bednar marked this pull request as ready for review April 27, 2022 06:11
@bednar bednar requested a review from rhajek April 27, 2022 06:11
@bednar bednar merged commit 90a4eaf into master Apr 28, 2022
@bednar bednar deleted the timeout branch April 28, 2022 08:24
@bednar bednar added this to the 4.2.0 milestone Apr 28, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants