Closed olivertzeng closed 5 months ago
一些意見跟疑問:
git push -f
),同時維持 PR 標題以及 commit 標題、 commit 內容一致。目前這樣用不停新增 commit、同時 commit 往往一次涵蓋多種類型增訂修改、甚至根本就只是 (partially) revert 前幾個 commit 的修改,對 reviewer 來說,很難判斷這個 PR 的完成度。一些意見跟疑問:
* 我先前的 review comment,想問的並不是你的「資料來源」,而是要問你製作這個 PR 的動機:為什麼要發這個 PR? * 承續以上問題,教育部的字典收納用字讀音,顯然遠大於小麥目前的字詞收錄。那麼你會選擇加入目前的這些字,所憑藉的判斷標準(英文說的 criteria)是什麼? * 要不要試試看先在 [issue tracker](https://github.com/openvanilla/McBopomofo/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc) 或[討論版](https://github.com/openvanilla/McBopomofo/discussions)上發表你對小麥詞庫的意見(「為什麼我認為這些字詞該加入、這些字詞讀音該這樣改」),看看別人怎麼回應?目前你這樣 PR 的作法,對 reviewer 來說,很難理解「這個 PR 能幫小麥用戶解決什麼問題」──丟一句「教育部簡編本異體字就是國語字典修訂本」的回應,然後放幾張超大螢幕截圖(直接給 link 不好嗎?),我們不懂確實的用意。 * 因為已經一來一往很多次,你也自己關閉、重開很多次 PR 了,我必須要求:請你了解一下 git rebase 等指令的用法,了解怎麼 amend/squash commit (+ `git push -f`),同時維持 PR 標題以及 commit 標題、 commit 內容一致。目前這樣用不停新增 commit、同時 commit 往往一次涵蓋多種類型增訂修改、甚至根本就只是 (partially) revert 前幾個 commit 的修改,對 reviewer 來說,很難判斷這個 PR 的完成度。
行數 1 除了燥以外其他都是隨機從簡編本中隨機挑選 所以我可能會revert 我的 commit
3 ok我會去查看的 4 抱歉造成麻煩 之前我有去試著學習看看 不過都失敗就放棄了 我會再嘗試看看的!我就先不關這個pr但會改成其他主題
@olivertzeng you can start with some GitHub blog posts, this is a good one. https://github.blog/2022-06-30-write-better-commits-build-better-projects/
教育部簡編本異體字就是國語字典修訂本 Sent from Proton Mail for iOS 於 週四, 3月 14, 2024 時間 10:35, Lukhnos Liu @.***> 寫道:
@lukhnos commented on this pull request.
In Source/Data/BPMFMappings.txt: > @@ -41248,7 +41248,7 @@ 肉搏 ㄖㄡˋ ㄅㄛˊ 肉末 ㄖㄡˋ ㄇㄛˋ 肉柱 ㄖㄡˋ ㄓㄨˋ -肉燥 ㄖㄡˋ ㄗㄠˋ
肉燥念作「ㄖㄡˋ ㄗㄠˋ」是非常普遍的用法,你是根據什麼決定刪除所有ㄗㄠˋ的讀音,改成ㄙㄠˋ的?
In Source/Data/BPMFBase.txt: > @@ -14200,6 +14203,7 @@ 鵩 ㄈㄨˊ fu2 zj6 big5 鶝 ㄈㄨˊ fu2 zj6 big5 沸 ㄈㄨˊ fu2 zj6 big5 +佛 ㄈㄨˊ fu2 zj6 big5
加入這個讀音的依據是?
In Source/Data/BPMFBase.txt: > @@ -684,6 +684,7 @@ 蔈 ㄅㄧㄠ biao 1ul big5 爂 ㄅㄧㄠ biao 1ul big5 藨 ㄅㄧㄠ biao 1ul big5 +嫑 ㄅㄧㄠˊ biao 1ul6 big5
加入這個字的依據是?我對其他新增字有相同問題。
In Source/Data/BPMFBase.txt: > @@ -13865,7 +13869,6 @@ 繰 ㄗㄠˇ zao3 yl3 big5 造 ㄗㄠˋ zao4 yl4 big5 皂 ㄗㄠˋ zao4 yl4 big5 -燥 ㄗㄠˋ zao4 yl4 big5
BPMFBase.txt 的資料是根據 bpmf.cin 建立的。我們原則上不刪除 bpmf.cin 收錄的字與讀音,也不改變 bpmf.cin 裡的順序: https://raw.githubusercontent.com/openvanilla/openvanilla/master/DataTables/bpmf.cin
— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>