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

awc: improve module documentation #1656

Merged
merged 7 commits into from Sep 9, 2020
Merged

Conversation

popzxc
Copy link
Member

@popzxc popzxc commented Sep 8, 2020

PR Type

Docs

PR Checklist

Check your PR fulfills the following:

  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • A changelog entry has been made for the appropriate packages.
  • Format code with the latest stable rustfmt

Overview

Subjectively, one great part of reqwest is its documentation, which provides usage examples for most common users' needs right on the main page of docs.rs.

This PR extends the `awc main doc-comment with the similar list of examples, so that users can see how to use library without digging deep into the crate docs (or even the source code).

@robjtede robjtede changed the title awc: Provide Reqwest-like crate doc-comment awc: improve module documentation Sep 8, 2020
@popzxc

This comment has been minimized.

@robjtede

This comment has been minimized.

awc/src/lib.rs Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Sep 9, 2020

Codecov Report

Merging #1656 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1656   +/-   ##
=======================================
  Coverage   53.35%   53.35%           
=======================================
  Files         117      117           
  Lines       11813    11813           
=======================================
  Hits         6303     6303           
  Misses       5510     5510           
Impacted Files Coverage Δ
awc/src/lib.rs 36.36% <ø> (ø)

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 9d05349...1125fe6. Read the comment docs.

awc/src/lib.rs Outdated Show resolved Hide resolved
awc/src/lib.rs Outdated Show resolved Hide resolved
awc/src/lib.rs Outdated Show resolved Hide resolved
awc/src/lib.rs Outdated Show resolved Hide resolved
awc/src/lib.rs Show resolved Hide resolved
popzxc and others added 2 commits September 9, 2020 14:04
Improve wording

Co-authored-by: Rob Ede <robjtede@icloud.com>
awc/src/lib.rs Outdated Show resolved Hide resolved
Copy link
Member

@robjtede robjtede left a comment

Choose a reason for hiding this comment

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

nice job, thanks 👍

@popzxc
Copy link
Member Author

popzxc commented Sep 9, 2020

Ah, Windows tests fail the same way as in #1654.

I'll restore URLs to HTTP and open an issue.

Edit: Actually, it's already reported: #1560

@robjtede
Copy link
Member

robjtede commented Sep 9, 2020

I'll restore URLs to HTTP

Fair enough. We'll merge with this and I shall try to remember to change them back to HTTPS before stable release. We should get this working on Windows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants