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

✏ Reword and clarify text in tutorialdocs/en/docs/tutorial/body-nested-models.md#5169

Merged
tiangolo merged 1 commit intofastapi:masterfrom
papb:patch-1
Aug 26, 2022
Merged

✏ Reword and clarify text in tutorialdocs/en/docs/tutorial/body-nested-models.md#5169
tiangolo merged 1 commit intofastapi:masterfrom
papb:patch-1

Conversation

@papb
Copy link
Contributor

It currently says"list of items", and since the entity in the example is calledItem, this was a little confusing for me. Avoiding the word"item" in this sentence is probably better.

GuildGamer reacted with thumbs up emoji
@github-actions
Copy link
Contributor

📝 Docs preview for commitb9fce4b at:https://62d5ecb7f767f50b8c5fc37a--fastapi.netlify.app

@codecov
Copy link

codecovbot commentedJul 18, 2022

Codecov Report

Merging#5169 (b9fce4b) intomaster (e35df68) willnot change coverage.
The diff coverage is100.00%.

@@            Coverage Diff            @@##            master     #5169   +/-   ##=========================================  Coverage   100.00%   100.00%           =========================================  Files          532       532             Lines        13684     13684           =========================================  Hits         13684     13684
Impacted FilesCoverage Δ
fastapi/__init__.py100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Copy link
Contributor

@edisnakeedisnake left a comment

Choose a reason for hiding this comment

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

looks good

Copy link
Contributor

@JarroVGITJarroVGIT left a comment

Choose a reason for hiding this comment

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

LGTM

@tiangolotiangolo changed the titleMinor clarification in body-nested-models tutorial✏ Reword and clarify text in tutorialdocs/en/docs/tutorial/body-nested-models.mdAug 26, 2022
@tiangolo
Copy link
Member

Great! Good point, thanks@papb! 🚀

papb reacted with heart emoji

@tiangolotiangolo merged commit7021346 intofastapi:masterAug 26, 2022
@papbpapb deleted the patch-1 branchAugust 31, 2022 21:33
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

2 more reviewers

@edisnakeedisnakeedisnake approved these changes

@JarroVGITJarroVGITJarroVGIT approved these changes

Reviewers whose approvals may not affect merge requirements

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

@papb@tiangolo@edisnake@JarroVGIT

Comments


[8]ページ先頭

©2009-2026 Movatter.jp