openvanilla / McBopomofo

小麥注音輸入法
http://mcbopomofo.openvanilla.org/
MIT License
624 stars 76 forks source link

新增有關化學字詞 #442

Closed olivertzeng closed 7 months ago

olivertzeng commented 8 months ago

some other notable stuff

olivertzeng commented 8 months ago

really sorry about the last pr chemist = 只有化學的東西

olivertzeng commented 8 months ago

謝謝。我針對 PR 內容留了評論。

我對 PR 形式上(也就是 PR / commit title 以及 PR 描述)有兩條意見:

  1. 請問標題 "Really chemistry only okay?" 的用意是?我不太明白在 PR / commit title 中出現疑問句,要傳達的意義是什麼。

  2. 請問 PR description 中,"unarguable chewing fixes from 簡編本" 的意思是什麼呢?我讀起來,這是三件不一樣的事。什麼是 "unarguable"? 我以為發送 PR 本身就已經是在提出意見,"to make an argument (that the developers should accept the proposed change)",也許是我不了解你的用意?此外,你提到酷音輸入法,是否你 PR 的修訂,是起源自酷音專案中的 PR 或討論?如果是這樣,是否能提供相關連結?如果「簡編本」指的是《國語辭典簡編本》,請問是否能擇要引用來源?

  1. 已修改
  2. 修改的讀音皆出自《國語辭典簡編本》所以不是像電子ㄗ˙的個人preference 修改
olivertzeng commented 8 months ago

謝謝。我針對 PR 內容留了評論。

我對 PR 形式上(也就是 PR / commit title 以及 PR 描述)有兩條意見:

  1. 請問標題 "Really chemistry only okay?" 的用意是?我不太明白在 PR / commit title 中出現疑問句,要傳達的意義是什麼。

  2. 請問 PR description 中,"unarguable chewing fixes from 簡編本" 的意思是什麼呢?我讀起來,這是三件不一樣的事。什麼是 "unarguable"? 我以為發送 PR 本身就已經是在提出意見,"to make an argument (that the developers should accept the proposed change)",也許是我不了解你的用意?此外,你提到酷音輸入法,是否你 PR 的修訂,是起源自酷音專案中的 PR 或討論?如果是這樣,是否能提供相關連結?如果「簡編本」指的是《國語辭典簡編本》,請問是否能擇要引用來源?

我有提到酷注音? 要是有那可能是我打錯字或是我常常commit message 都會亂打

olivertzeng commented 8 months ago

我已回覆所有評論 等等會修改 @lukhnos

olivertzeng commented 8 months ago

我已經完成幾乎所有更改以及不太確定是否 revert commit 的字詞

olivertzeng commented 8 months ago
  1. 請問 PR description 中,"unarguable chewing fixes from 簡編本" 的意思是什麼呢?[…]此外,你提到酷音輸入法,是否你 PR 的修訂,是起源自酷音專案中的 PR 或討論?[…] 我有提到酷注音? 要是有那可能是我打錯字或是我常常commit message 都會亂打

如附圖,你的英文敘述是 "unarguable chewing fixes",此處 chewing 如果指的不是新酷音輸入法專案,請問意思是?

image

抱歉我以為 chewing 是指注音不是酷注音 aka 我以為 chewing == bpmf 我會試試看在我的local machine 上使用詞 也會補上 phrase.occ

最後,目前你是用新增 commit 的方式回應 PR review comments(這是 ok 的),但第一個 commit 的 commit message 仍是 Really chemistry only okay? 因此,請將這幾個 commit 給 squash 成一個,趁此機會將 commit message 改為符合這個 PR 用意的敘述。 我不會弄ㄟ 但我會去查查看怎麼用

olivertzeng commented 8 months ago

已完成修改(除了squash commit以外) @lukhnos

olivertzeng commented 7 months ago

close

lukhnos commented 7 months ago

close

其實可以透過 git rebase -i + git push -f 等指令來達成 squash commits、修訂 commit 內容的目的。既然你開了 #444,這一點僅作為日後回應 PR review comments 的建議,供你參考。

olivertzeng commented 7 months ago

Ok Sent from Proton Mail for iOS 於 週六, 3月 2, 2024 時間 20:42, Lukhnos Liu @.***> 寫道:
close

其實可以透過 git rebase -i + git push -f 等指令來達成 squash commits、修訂 commit 內容的目的。既然你開了 #444,這一點僅作為日後回應 PR review comments 的建議,供你參考。

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you modified the open/close state.Message ID: @.***>