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

🌐 Add French translation fordocs/fr/docs/benchmarks.md#2155

Merged
tiangolo merged 7 commits intofastapi:masterfrom
clemsau:master
Jul 27, 2023
Merged

🌐 Add French translation fordocs/fr/docs/benchmarks.md#2155
tiangolo merged 7 commits intofastapi:masterfrom
clemsau:master

Conversation

@clemsau
Copy link
Contributor

Hello 👋,

This is a PR to translatedocs/benchmarks.md to French, as told in the issue#1972.

Sorry for the typo in the commit message.

See the formatted documenthere

Thanks for those who take the time to validate it.

tiangolo reacted with rocket emoji
@codecov
Copy link

codecovbot commentedOct 10, 2020•
edited
Loading

Codecov Report

Patch and project coverage have no change.

Comparison is base(cf73051) 100.00% compared to head(68ff943) 100.00%.

❗ Current head68ff943 differs from pull request most recent head4da8b7c. Consider uploading reports for the commit4da8b7c to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@##            master     #2155     +/-   ##===========================================  Coverage   100.00%   100.00%             ===========================================  Files          540       239    -301       Lines        13969      7079   -6890     ===========================================- Hits         13969      7079   -6890

see 356 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment?Let us know in this issue.

@github-actions
Copy link
Contributor

📝 Docs preview for commitb97ede6 at:https://5f81fa2b1433d5fc03f80016--fastapi.netlify.app

@Jefidev
Copy link
Contributor

Hi, I reviewed your text and for me it is ok. I just have a suggestion :
I think that it is prettier to change this sentences :
[...] les applications FastAPI tournant sous Uvicorn commeun des frameworks Python les plus rapides disponibles,

to this :
[...] les applications FastAPI tournant sous Uvicorn commeétant l'un des frameworks Python les plus rapides disponibles,

@Jefidev
Copy link
Contributor

Oh I didn't notice at first glance but you forgot to update themkdocs.yml.

@clemsau
Copy link
ContributorAuthor

I have taken into account your proposal@Jefidev , and I updated themkdocs.yml .

@github-actions
Copy link
Contributor

📝 Docs preview for commit68ff943 at:https://5f8240142c8e2f89a71bff63--fastapi.netlify.app

@rjNemo
Copy link
Contributor

I left a few comments. To me it would improve the translation. However I think this PR is good as it is.

Jefidev reacted with heart emoji

* Comme Starlette, FastAPI utilise Uvicorn et ne peut donc pas être plus rapide que ce dernier.
* FastAPI apporte des fonctionnalités supplémentaires à Starlette. Des fonctionnalités qui sont nécessaires presque systématiquement lors de la création d'une API, comme la validation des données, la sérialisation. En utilisant FastAPI, on obtient une documentation automatiquement (qui ne requiert aucune manipulation pour être mise en place).
* Si on n'utilisait pas FastAPI mais directement Starlette (ou un outil équivalent comme Sanic, Flask, Responder, etc) il faudrait implémenter la validation des données et la sérialisation par nous-même. Le résultat serait donc le même dans les deux cas mais du travail supplémentaire serait à réaliser avec Starlette, surtout en considérant que la validation des données et la sérialisation représentent la plus grande quantité de code à écrire dans une application.
* De ce fait, en utilisant FastAPI on minimise le temps de développement, les bugs, le nombre de lignes de code, et on obtient les mêmes performances (si ce n'est de meilleurs performances) que l'on aurait pu avoir sans ce framework (en ayant à implémenter de nombreuses fonctionnalités importantes par nous même).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* De ce fait, en utilisant FastAPI on minimise le temps de développement, les bugs, le nombre de lignes de code, et on obtient les mêmes performances (si ce n'est de meilleurs performances) que l'on aurait pu avoir sans ce framework (en ayant à implémenter de nombreuses fonctionnalités importantes par nous même).
* De ce fait, en utilisant FastAPI on minimise le temps de développement, les bugs, le nombre de lignes de code, et on obtient les mêmes performances (si ce n'est de meilleurs performances) que l'on aurait pu avoir sans ce framework (en ayant à implémenter de nombreuses fonctionnalités importantes par nous-mêmes).

@tiangolotiangolo changed the title 📝 Add French translation for docs/benchmarks.md 📝 Add French translation fordocs/fr/docs/benchmarks.mdJun 26, 2023
@tiangolotiangolo changed the title 📝 Add French translation fordocs/fr/docs/benchmarks.md🌐 Add French translation fordocs/fr/docs/benchmarks.mdJun 26, 2023
@tiangolo
Copy link
Member

📝 Docs preview for commit7e55a1f at:https://6499cda889d3ab086b01d304--fastapi.netlify.app

Co-authored-by: Sam Courtemanche <smlep.pro@gmail.com>Co-authored-by: Ruidy <ruidy.nemausat@gmail.com>
@tiangolo
Copy link
Member

📝 Docs preview for commit1d30a7e at:https://64b4ffeeb4ec054a27d6c0d8--fastapi.netlify.app

@tiangolo
Copy link
Member

📝 Docs preview for commit4da8b7c at:https://64b652e4723d8f44d0b9ed18--fastapi.netlify.app

@tiangolo
Copy link
Member

Thank you@clemsau! 🍰

And thanks for the reviews@Smlep,@emmrichard,@axel584,@rjNemo ☕

clemsau reacted with hooray emoji

@tiangolotiangolo merged commit3ffebbc intofastapi:masterJul 27, 2023
erosennin pushed a commit to erosennin/fastapi that referenced this pull requestJul 31, 2023
Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>Co-authored-by: Sam Courtemanche <smlep.pro@gmail.com>Co-authored-by: Ruidy <ruidy.nemausat@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

4 more reviewers

@SmlepSmlepSmlep left review comments

@emmrichardemmrichardemmrichard approved these changes

@axel584axel584axel584 approved these changes

@rjNemorjNemorjNemo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@clemsau@Jefidev@rjNemo@tiangolo@emmrichard@axel584@Smlep

Comments

[8]ページ先頭
©2009-2026 Movatter.jp