openvanilla / McBopomofo

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

Chemistry #441

Closed olivertzeng closed 6 months ago

olivertzeng commented 6 months ago

chemistry first+some math stuff for nerds lol

olivertzeng commented 6 months ago

感謝提出建議 其他新增的辭世我忘記刪除的詞 明天我會刪除其他out of scope 的詞 Math stuff like 鎖鏈率

olivertzeng commented 6 months ago

謝謝你製發新的 PR。我在先前 PR 的回應中提到,建議把 PR 修改為「把範圍侷限現化學相關字詞,並且把目前諸多 commit 透過 rebase 修改/squash 成一個單一的、小規模的、附簡單說明(寫中文就可以)的 commit」,但目前的 PR 明顯文題不符:

* PR 標題為 Chemistry(意思是?寫中文標題是否比較容易澄清 PR 的意圖?),但是 commits 中包含了其他詞類。例如:增加「液」相關字詞的語音。關於這一點,我要求另開 PR 或至少開新 issue 討論。其他 emoji 的修訂明顯超出範圍,其中也有先前 @mjhsieh 說拒收的修訂。

* 增加諸多「子」相關字詞讀音,先前已經提過,仍有待討論。這個明顯超過目前 PR 所說範圍。

* PR 內容提到 "some math stuff",請問增加了哪些數學名詞?「因子」一類的修訂,我認為只能算在「子」讀音增修上。不管怎麼說,這仍然超出了 PR 標題所說的範圍。

* 目前的 PR 有許多 commit,commit message 並沒有簡短說明該 commit 用意。以修訂詞彙的 PR 來說,太多 commit 也不成比例。請在修改完後,用 git 將諸多 commit 給 squash 一個單一的 commit。

對了我好像忘記 git pull 對不起 難怪會有這個誤會