Codecov Report

Merging #1973 (2106e4c) into master (e35df68) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master     #1973   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          532       532           
  Lines        13684     13684           
=========================================
  Hits         13684     13684           

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 50fb34b...2106e4c. Read the comment docs.

📝 Docs preview for commit 1c22c9683295482146565bcdafb959ac3ea3b876 at: https://5f46199400610924347650bb--fastapi.netlify.app

@JulianMaurin I suggest you to create a separate MR for the initial language setup. This was done on the other languages because the setup itself doesn't need a French speaker as reviewer and it can be merged faster.

@JulianMaurin I suggest you to create a separate MR for the initial language setup. This was done on the other languages because the setup itself doesn't need a French speaker as reviewer and it can be merged faster.

Hey @Kludex, thanks for your feedback.
Here is the PR: #1975

Thanks to Development - Contributing' guidelines:

1. Translation is correct for me ( Thx Julian ;) )
2. Translation can be merged! ( Thx to FastAPI Team ;) )

Thanks in advance.

📝 Docs preview for commit e59e2f0 at: https://5f79e684684027c1fbceaba1--fastapi.netlify.app

Hi,

Thank you for the great work!
I've made some suggestions to the text. I hope it was the right method 😅 and that it will be useful.

There is still missing space after colon ponctuation, like for example line 36 to 42. But I think I've already made too much suggestions.
I would also be favorable to put a space after the three dots as in the expression '... de' (line 383). But I wasn't sure at 100%...

I would like to contribute from time to time. Let me know what the best method to help when reviewing the translation.

Thanks!

📝 Docs preview for commit 3145273 at: https://62d325677fc77351ad174235--fastapi.netlify.app

📝 Docs preview for commit fe9737d at: https://62d329558083e155772ea969--fastapi.netlify.app

📝 Docs preview for commit 93475d2 at: https://62d32fa481fa5c5b79b8a7d6--fastapi.netlify.app

📝 Docs preview for commit 51a21a0 at: https://62d3346943a4525bf5973154--fastapi.netlify.app

@jburckel

I've made some suggestions to the text. I hope it was the right method 😅 and that it will be useful.

Yes was, thanks 🙏

There is still missing space after colon ponctuation, like for example line 36 to 42. But I think I've already made too much suggestions.

✅ 93475d2

Validating there are no more missing withspaces using the following expressions:

• before colon: [^( |https|text/-align)]:
• after colon: :[^( |\/\/|15px|app|8)]

I would also be favorable to put a space after the three dots as in the expression '... de' (line 383). But I wasn't sure at 100%...

✅ 51a21a0

According to LeConjugeur you are right.

Validating using the following expressions:

• additional withspaces before ellipsis: \.\.\.
• missing withspaces after ellipsis: \.\.\.[^( |\])]

I would like to contribute from time to time. Let me know what the best method to help when reviewing the translation.

Reviewing PRs is great! You can also take a look at the documentation and at #1972 to add you own translations :)

📝 Docs preview for commit 1696767 at: https://62d33a4104222759faaa8464--fastapi.netlify.app

It seems this was translated at some point in another PR. So I'll close this one, but thanks for the effort! ☕