Edited at

WIP: GitHub 頻出英語フレーズ 100

※ WIP なのでまだ 100 個ないです。随時書き足していきます

OSS に Pull Request を投げるときや、逆に自分がメンテナーをしている repo のレビューやコメントのやりとりに使える文例集です。リーディングではなくライティングに活用することを意図しています。主に rails/rails のコメント欄などを参考にしています。

Travis がコケてますが、私の変更とは関係ありません

Failures in TravisCI aren't related to my changes.

関連する Issue と紐付けるとき

Related to #30101


Looks like many existing Rails users are going to experience this problem.


Will do, I'll link the pull request here when I create it.

コーディング規約とかで禁止するときは Disallow が便利

Disallow specific global variables

動きがないので close します。必要であれば reopen してください

Closing due to inactivity. Reopen as needed.


Occasionally failing tests

重複した Issue や PR を閉じるとき

Closing in favor of #26743.

A を使っている場合は B は不要です

If you are using heroku.yml as your build manifest, a Procfile is not required.


Remove unused initialize_attributes method

development モードだと正常に動く理由は

The reason why it works in development is


Consider the following code:

不必要な複雑さをもたらす gem

a gem which can lead to unnecessary complexities.

諸事情で Approve できないときの文例

hmm, I can't 👍 on this because my application uses it.


Not yet.


I've run into an issue that I think is caused by this.


Both are needed.

もし hoge メソッドを使用している場合だったらどうなりますか?

What if we use the hoge method?


in the same vein that it does with Turbolinks


Update Yard dependency (CVE-2017-17042)


You need to move the event.waitUntil call at the beginning of the function, otherwise your service worker could be killed in the meantime (which is what's happening).


With request specs, you can:

- specify a single request

- specify multiple requests across multiple controllers