📝

[備忘録]初めてGithubでPull-Requestを送ってみた

2 min read

おはこんばんにちわ

先日こんな記事を書きました

https://zenn.dev/tantan_tanuki/articles/5621081816b633

この記事でビルドしたDockerイメージが10G位あるって話をしたんですがDockerfileを見てみると改善の余地がありそうだったのでちょろっと弄って見た結果3.5G位容量が縮まりました

$ docker images voicevox_core*
REPOSITORY           TAG       IMAGE ID       CREATED        SIZE
voicevox_core_test   latest    c7bc4a4dc9cd   2 hours ago    6.22GB
voicevox_core        latest    dfd96042c88c   24 hours ago   9.73GB

今までPull-Requestをした経験が無かったので勉強も兼ねて今回の修正内容をPRしてみた結果
無事取り込んでもらえたので今後どこかにPRする時用の備忘録として残します

まずはfork

自身のgithubアカウントにログインして

https://github.com/Hiroshiba/voicevox_core

の画面右上のforkを押して自分のリポジトリに取り込みます

forkしたソースをローカルにclone

ローカルにgithubからcloneして取り込みます

$ git clone https://github.com/tantan-tanuki/voicevox_core.git

Pull-Request用のbranchを切る

$ git checkout -b develop

こういう時のブランチ名って何が一般的なんだろう。。。
mainにそのまま修正加えてPRでも良かったのかな。。。。

ブランチで変更した内容をコミット

$ git add .
$ git commit -m'修正した内容のコメント'

push

$ git push -u origin develop

Pull-Requestを作成

githubにログインしてforkしたリポジトリを開くと
「Compare & pull request」
ボタンが表示されているのでクリック

取り込み先と取り込み元のブランチが正しいことを確認したら
修正内容を書いて「Create pull request」をクリックすると作成される

Imgur

レビュー結果を反映する

今回Dockerfileの位置を変更したのですが元の位置に戻して欲しいというレビュー結果が返ってきたので対応したものを再度pushして返信しました

Imgur

マージ完了

しばらくすると問題ないとのことでマージしてもらえました!

Imgur

後片付け

さてPRを受け付けてもらうためだけにforkしたので不要となったリポジトリを削除します

Imgur

終了

というわけで無事取り込んでもらうことが出来ました🎉

↓は実際に作成されたPull-Request

https://github.com/Hiroshiba/voicevox_core/pull/9

大したことしてないのにContributorsの欄に自分が載ってるとむず痒いですね😅

Dockerfileとか完全に独学で人に見せるのも初めてだったのでドキドキしましたが
すんなり受け入れてもらえて安心しました

これで

😤「俺は窓の杜に掲載されるようなOSSプロジェクトに貢献したことがあるんやで」

みたいな事が言えるようになったので

副産物として

IT音痴の新人にのみ効果を発揮する威嚇スキルを手に入れることが出来ました🤭

それでは良いGitライフを〜

しゃみしゃっきりー

P.S.

全然記事の内容と関係ないのですが記事にするほどでも無いのでチラ裏的にここに書きます

Java17がまた商用利用無料となったようですね

https://www.publickey1.jp/blog/21/oracle_jdkjava_17.html

Oracleに買収された時点で有償化されるのは見えていたのですが2018年にやはり有償化されました

私は当時予算のない現場がEOL超えた状態で現行のまま使い続けるという決断をしたのを目の当たりにしています

そのシステムは今も元気に動いていたりします

しかし各社が無償のJDKを提供し始めて誰も使ってくれなくなった途端にこの方針転換はどうなんでしょう。

しれっとLTSの期間も短縮しようとしてるし。。

当時痛みを伴って決別した人たちの神経を逆撫でしてるだけなような。。。

Discussion

ログインするとコメントできます