Qiita Teams that are logged in
You are not logged in to any team

Log in to Qiita Team
Community
OrganizationAdvent CalendarQiitadon (β)
Service
Qiita JobsQiita ZineQiita Blog
41
Help us understand the problem. What is going on with this article?
@ttanimichi

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

※ 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?

41
Help us understand the problem. What is going on with this article?
Why not register and get more from Qiita?
  1. We will deliver articles that match you
    By following users and tags, you can catch up information on technical fields that you are interested in as a whole
  2. you can read useful information later efficiently
    By "stocking" the articles you like, you can search right away

Comments

No comments
Sign up for free and join this conversation.
Sign Up
If you already have a Qiita account Login
41
Help us understand the problem. What is going on with this article?