Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

pencil2: Fix awkward pronoun usage#1824

Merged
tiangolo merged 3 commits intofastapi:masterfrom
Mause:mend-pronoun-usage
Aug 9, 2020
Merged

pencil2: Fix awkward pronoun usage#1824
tiangolo merged 3 commits intofastapi:masterfrom
Mause:mend-pronoun-usage

Conversation

@Mause
Copy link
Contributor

No description provided.

Edward-Knight reacted with thumbs up emojirkbeatss and dferreruw reacted with heart emoji
@codecov
Copy link

codecovbot commentedAug 2, 2020
edited
Loading

Codecov Report

Merging#1824 intomaster willnot change coverage.
The diff coverage isn/a.

Impacted file tree graph

@@            Coverage Diff             @@##            master     #1824    +/-   ##==========================================  Coverage   100.00%   100.00%            ==========================================  Files          235       239     +4       Lines         6831      7124   +293     ==========================================+ Hits          6831      7124   +293
Impacted FilesCoverage Δ
fastapi/params.py100.00% <0.00%> (ø)
fastapi/routing.py100.00% <0.00%> (ø)
tests/test_path.py100.00% <0.00%> (ø)
fastapi/encoders.py100.00% <0.00%> (ø)
tests/test_query.py100.00% <0.00%> (ø)
fastapi/exceptions.py100.00% <0.00%> (ø)
fastapi/applications.py100.00% <0.00%> (ø)
tests/test_fakeasync.py100.00% <0.00%> (ø)
fastapi/security/http.py100.00% <0.00%> (ø)
tests/test_application.py100.00% <0.00%> (ø)
... and72 more

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatefc494e3...652c655. Read thecomment docs.

@MauseMause marked this pull request as ready for reviewAugust 2, 2020 11:23
@github-actions
Copy link
Contributor

📝 Docs preview for commit976d35f at:https://5f26add9ff928d3432cb9603--fastapi.netlify.app

Copy link
Member

@KludexKludex left a comment

Choose a reason for hiding this comment

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

Nice catch! 🎉

Mause reacted with heart emoji
@github-actions
Copy link
Contributor

📝 Docs preview for commit505c480 at:https://5f27d491ff928debb1cb91ae--fastapi.netlify.app

@MauseMause mentioned this pull requestAug 7, 2020
2 tasks
@connebs
Copy link

connebs commentedAug 7, 2020
edited
Loading

A number of these are talking about "the client". Is that not more of an "it"?

@tiangolo
Copy link
Member

Thanks a lot! 🔍 📝

I want to have a welcoming space in the FastAPI community and I don't want to have language that might feel unwelcoming. 🚀

At the same time, one of my main objectives in the docs is to reduce ambiguity to make it as easy to learn and understand as possible, even for people that might not be native English speakers (like myself), so I updated those examples a bit to make them non-gender specific without having to deal with the ambiguity of possible plurals.

@tiangolotiangolo merged commit0a2fc78 intofastapi:masterAug 9, 2020
@tiangolotiangolo changed the title✏️ Fix awkward pronoun usagepencil2: Fix awkward pronoun usageFeb 24, 2023
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@KludexKludexKludex approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Mause@connebs@tiangolo@Kludex

Comments


[8]ページ先頭

©2009-2026 Movatter.jp