Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit e992920

Browse files
author
Nils Lindemann
committed
Remove redundant explanation and restructure
Some things will be re-added (and explained better) in the following commit.
1 parent 6668828 commit e992920

File tree

1 file changed

+5
-13
lines changed

1 file changed

+5
-13
lines changed

docs/en/docs/contributing.md

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,9 @@ Here are the steps to help with translations.
255255

256256
#### Tips and guidelines
257257

258-
* Check the currently <a href="https://github.com/tiangolo/fastapi/pulls" class="external-link" target="_blank">existing pull requests</a> for your language and add reviews requesting changes or approving them.
258+
* Check the currently <a href="https://github.com/tiangolo/fastapi/pulls" class="external-link" target="_blank">existing pull requests</a> for your language. You can filter the PRs by the ones with the label for your language. For example, for Spanish, the label is <a href="https://github.com/tiangolo/fastapi/pulls?q=is%3Aopen+sort%3Aupdated-desc+label%3Alang-es+label%3Aawaiting-review" class="external-link" target="_blank">`lang-es`</a>.
259+
260+
* Add reviews requesting changes or approving them. For the languages I don't speak, I'll wait for several others to review the translation before merging.
259261

260262
!!! tip
261263
You can <a href="https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/commenting-on-a-pull-request" class="external-link" target="_blank">add comments with change suggestions</a> to existing pull requests.
@@ -264,19 +266,9 @@ Here are the steps to help with translations.
264266

265267
* Check if there's a <a href="https://github.com/tiangolo/fastapi/discussions/categories/translations" class="external-link" target="_blank">GitHub Discussion</a> to coordinate translations for your language. You can subscribe to it, and when there's a new pull request to review, an automatic comment will be added to the discussion.
266268

267-
* Add a single pull request per page translated. That will make it much easier for others to review it.
268-
269-
For the languages I don't speak, I'll wait for several others to review the translation before merging.
270-
271-
* You can also check if there are translations for your language and add a review to them, that will help me know that the translation is correct and I can merge it.
272-
* You could check in the <a href="https://github.com/tiangolo/fastapi/discussions/categories/translations" class="external-link" target="_blank">GitHub Discussions</a> for your language.
273-
* Or you can filter the existing PRs by the ones with the label for your language, for example, for Spanish, the label is <a href="https://github.com/tiangolo/fastapi/pulls?q=is%3Aopen+sort%3Aupdated-desc+label%3Alang-es+label%3Aawaiting-review" class="external-link" target="_blank">`lang-es`</a>.
274-
275-
* Use the same Python examples and only translate the text in the docs. You don't have to change anything for this to work.
276-
277-
* Use the same images, file names, and links. You don't have to change anything for it to work.
269+
* If you translate a page, add a single pull request per page translated. That will make it much easier for others to review it.
278270

279-
* To check the 2-letter code for the language you want to translate you can use the table <a href="https://en.wikipedia.org/wiki/List_of_ISO_639-1_codes" class="external-link" target="_blank">List of ISO 639-1 codes</a>.
271+
* To check the 2-letter code for the language you want to translate, you can use the table <a href="https://en.wikipedia.org/wiki/List_of_ISO_639-1_codes" class="external-link" target="_blank">List of ISO 639-1 codes</a>.
280272

281273
#### Existing language
282274

0 commit comments

Comments
 (0)