関数の命名法をPython風に & fragをflagに

[Naohiro2g]

はい、issueの練習です。

関数の命名法、大体は、Python風のスネイクケースになっていますが、Java風のキャメルケースが一部混じっているので、統一しましょうか。MinecraftがJavaで、APIもJava風なので、引っ張られがちですけど。

convert_mcje.pyのcellOpen()、raiseFrag()、dropFrag()ですね。
定義側

minecraft_remote/convert_mcje.py

Line 53 in 085175a

def cellOpen(self,j,i):

minecraft_remote/convert_mcje.py

Line 81 in 085175a

def raiseFrag(self,j,i):

minecraft_remote/convert_mcje.py

Line 86 in 085175a

def dropFrag(self,j,i):

参照側、mcje_sweeper.pyは、

minecraft_remote/mcje_sweeper.py

Line 268 in 085175a

self.mjs.cellOpen(j, i)

minecraft_remote/mcje_sweeper.py

Line 217 in 085175a

self.mjs.raiseFrag(j, i)

minecraft_remote/mcje_sweeper.py

Line 227 in 085175a

self.mjs.dropFrag(j, i)

ちなみに、fragは、flagが正しいのでついでに修正しましょう。issueは細かく分けたほうが良いですが、まぁ、このぐらいは一緒で。ちなみに、issueは同じでもコミットは分けてプルリクも分ける、もあり。コミットも分けるけどプルリクは一緒、もあり。コミットは分けておきましょう。

修正分をコミットする時に、「#」を書くと、このissueにリンクすることができます。

自分で修正してプルリクを発行、自分でマージする練習をしてみましょう。

1. mainから、ブランチfix-01を切る。
   • VS Codeで、左下のmain*って書いてあるところをクリック。
   • +新しい分岐の作成… で、fix-01と書いてエンター。
2. fix-01の状態で、ファイルを修正、ステージング、コミット、プッシュ。GitHub.comに新しいブランチができます。
3. 新しいブランチからプルリク作る？という[Open pull request]ボタンが表示されているので、押す。
4. fix-01からmainへのプルリクが発行される。
5. 自分でmainにマージ。fix-01を消去するボタンを押してコミット。fix-01消える。
6. VS Codeで、手元にプルしてmainが更新されているのを確認。

こんな感じで。