2
1

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 3 years have passed since last update.

明日もう一度ここに来てください。本物のプルリクというものをお見せしますよ

Last updated at Posted at 2021-10-20

プルリク

宣伝

Homebrew博士(自称)の私が「日本語で」質問に答えるためのDiscordサーバーを作りました!
Formulaの書き方がわからないとか、新しいCaskを追加したいといった質問に答えられます、是非いらしてください!

Diff

スクリーンショット 2021-10-20 12.20.10.png

経緯

  1. Homebrewのメンテナになりたい!
  2. Mike McQuaidにTwitterで突撃
  3. 「"help wanted"に貢献してね❤️」
  4. 最新の"help wanted"課題にこれを実装しますとコメント
  5. 20日経過

作業

  1. 頑張って状況再現する
  2. --debugフラグ付けてスタックトレースを取る
  3. 順番にスタックトレースを辿っていった結果、キャッチされるべき例外がキャッチされずにプログラムが異常終了していることに気づく
  4. Errno::ENOTEMPTYをキャッチするように変更(18文字)

まとめ

やろうと思えば問題解決のために100行近く新規のコードを追加することもできたかもしれません。
最初は、独自の例外ハンドリング機構を実装する必要があるかなと思っていました。
でも、それはスマートではありません。
「既存の資産をいかに上手に活用し、綺麗に問題を解決するか」がOSSへのプルリクの本質なのです。
たった18文字でも何物にも変えがたい素晴らしい貢献です。

2
1
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
2
1

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?