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

feat(translate): wsgiref.po#612

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mattwang44 merged 4 commits intopython:3.12fromneo-maker-sudo:feat/wsgiref
Aug 29, 2023

Conversation

neo-maker-sudo
Copy link
Contributor

Changed Description

  • Add partial document translation from english to traditional chinese

@@ -153,13 +186,19 @@ msgid ""
"*`` variables. It only supplies default values, and does not replace any "
"existing settings for these variables."
msgstr ""
"這個程式新增 WSGI 所需的各種參數，包括 ``HTTP_HOST``，``SERVER_NAME``，``SERVER_PORT``，``REQUEST_METHOD``"
"，``SCRIPT_NAME``，``PATH_INFO``，以及所有 :pep:`3333``\\ 定義的 ``wsgi.*`` 變數，它只提供預設值，並且不會取代"
"現有的這些變數設定。"

#: ../../library/wsgiref.rst:118
msgid ""
"This routine is intended to make it easier for unit tests of WSGI servers "
Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

@mattwang44 想詢問 matt 大，比如說這邊的 routine，不太瞭解主要是指什麼意思

Copy link
Collaborator

Choose a reason for hiding this comment

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

原文在此
image

Update environ... 說明這個 function 的功能為何，而後面兩個段落都以This routine... 開頭分別補充說明作用細節與目的。這邊以routine 來代指這個 function，就也暗示其為「慣例」、「例行公事」、「慣用程序」，意即因為測試上時常要更新 environ 而將這些步驟封裝成了這個 function。

Copy link
ContributorAuthor

@neo-maker-sudoneo-maker-sudoAug 23, 2023•
edited
Loading

Choose a reason for hiding this comment

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

喔喔，大概瞭解，所以意思是因為測試的時候需要更新environ，而這個動作又時常被調用，所以直接封裝起來，並使用routine 這個詞來代指 function，我可以這樣理解嗎？

Copy link
Collaborator

Choose a reason for hiding this comment

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

我的理解是這樣沒錯～

Copy link
ContributorAuthor

@neo-maker-sudoneo-maker-sudoAug 23, 2023•
edited
Loading

Choose a reason for hiding this comment

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

感謝大大解惑，有種豁然開朗的感覺 XD

mattwang44 reacted with laugh emoji
@mattwang44
Copy link
Collaborator

@neo-maker-sudo CI 檢查沒過，就先讓你嘗試看能否判讀 logs 說哪裡有問題囉

@neo-maker-sudo
Copy link
ContributorAuthor

neo-maker-sudo commentedAug 23, 2023•
edited
Loading

@mattwang44 我剛剛有看了，在 translate :pep:3333 是無效的 PEP number 而報錯，但 rst 格式沒錯，其他地方引用相同內容也沒問題，在想是不是因為. 後面接兩個空格導致的？我需要改原文嗎

@mattwang44
Copy link
Collaborator

@neo-maker-sudo 是格式問題歐，再檢查一下:pep:`3333`` 那邊有什麼問題

@neo-maker-sudo
Copy link
ContributorAuthor

neo-maker-sudo commentedAug 23, 2023•
edited
Loading

喔喔，我看錯地方拉 XD，眼睛脫窗了，想順便詢問一下 matt 大，就是有關 commit amend 調整並推送到遠端庫，我如果使用 push 覆寫的時候給 --force-with-lease 這件事情是 ok 的嗎

@mattwang44
Copy link
Collaborator

另外你的譯文每列長度看起來是自己抓的，希望要使用 powrap 做格式化

pip install powrappowrap library/wsgiref.po
neo-maker-sudo reacted with eyes emoji

@neo-maker-sudo
Copy link
ContributorAuthor

好的，那我可否在整篇文章翻譯完後再 format 嗎

mattwang44 reacted with thumbs up emoji

@mattwang44
Copy link
Collaborator

mattwang44 commentedAug 23, 2023•
edited
Loading

有關 commit amend 調整，我如果使用 push 覆寫的時候給 --force-with-lease 這件事情是 ok 的嗎

希望是讓 reviewer 的負擔輕一點：

  • reviewer 要 review 前都可以 amend + force push
  • 根據 reviewer comment 所做的更動希望是獨立的 commit，如此一來 reviewer 就可以比較不費力地看出你是改了哪些地方
    • 若是想要維持 commit history 乾淨的話，可以使用git commit --fixup <hash> 來補上後續更正的 commits，reviewer approve 後再git rebase -i <hash> --autosquash

我在平常開發上也都是盡量遵守這個原則

可否在整篇文章翻譯完後再 format 嗎

沒有問題！

P.S. 另有志工在嘗試實作 powrap 的 pre-commit (#608)，希望以後就不用自己多下指令處理~

neo-maker-sudo reacted with hooray emoji

@neo-maker-sudo
Copy link
ContributorAuthor

neo-maker-sudo commentedAug 23, 2023•
edited
Loading

取決於和 reviewer 互動，希望是讓 reviewer 的負擔輕一點

好的，瞭解了，我沒有用過--fixup 跟--autosquash，感覺很猛，之後再用專案自己測試看看好了，學了很多東西 XD

P.S. 另有志工在嘗試實作 powrap 的 pre-commit (https://github.com/python/python-docs-zh-tw/pull/608)，希望以後就不用自己多下指令處理~

好的，我去瞭解一下，感謝

mattwang44 reacted with thumbs up emojimattwang44 reacted with rocket emoji

Copy link
Collaborator

@mattwang44mattwang44 left a comment•
edited
Loading

Choose a reason for hiding this comment

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

補了一個 commit 小修標點符號，其他 LGTM!
我會先 merge PR，formatting 就之後再做沒關係，辛苦了～

neo-maker-sudo reacted with eyes emoji
@mattwang44mattwang44 merged commit6218509 intopython:3.12Aug 29, 2023
@neo-maker-sudo
Copy link
ContributorAuthor

好的，那我之後再繼續推上來後面的翻譯，感謝

beccalzh pushed a commit to beccalzh/python-docs-zh-tw that referenced this pull requestSep 4, 2024
* docs(wsgiref): add partial document translation from english to traditional chinese* docs(wsgiref): modified partial document translation content from rst:2 to rst:118* docs(wsgiref): fixed formatting issue at rst:112* Update library/wsgiref.po---------Co-authored-by: Wei-Hsiang (Matt) Wang <mattwang44@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@mattwang44mattwang44mattwang44 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.

2 participants
@neo-maker-sudo@mattwang44
[8]ページ先頭
©2009-2025 Movatter.jp