※ WIP なのでまだ 100 個ないです。随時書き足していきます
OSS に Pull Request を投げるときや、逆に自分がメンテナーをしている repo のレビューやコメントのやりとりに使える文例集です。リーディングではなくライティングに活用することを意図しています。主に rails/rails のコメント欄などを引用していますが、その他の様々な repo や Stack Overflow などからも拝借しています。
Travis がコケてますが、私の変更とは関係ありません
Failures in TravisCI aren't related to my changes.
動きがないので close します。必要であれば reopen してください
Closing due to inactivity. Reopen as needed.
まれにコケるテスト
Occasionally failing tests
関連する Issue と紐付けるとき
Related to #30101
自明な主語を省略するパターンその1
Looks like many existing Rails users are going to experience this problem.
自明な主語を省略するパターンその2
Will do, I'll link the pull request here when I create it.
コーディング規約とかで禁止するときは Disallow が便利
Disallow specific global variables
重複した 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
この点に関して、ドキュメントが少し不明瞭ですね
The documentation is a bit unclear on this.
OSS のバグを踏んで、その修正を Pull Request してる人がいて既に Merge もされているんだけど、その修正を含むバージョンがまだリリースされていないのでコメント欄で「リリースまだ？」って突きたいけど、メンテナの人を怒らせないよう最大限やわらかい表現で言いたいとき
Are you planning to release this in the next version?
下記の手順で解決しました
I was able to resolve by the following steps:
- foo
- bar
軽微なバグの修正
Minor bug fixes