1
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

Rails:PR前のセルフレビューチェックリスト メモ

Posted at

内容

  • Rubocopに引っかからない部分をチェックする

動作環境

  • ruby 3.3.0
  • Rails 7.1.3.3

使用しているもの

  • haml-rails gem

チェックリスト

link_toの書き方(haml)

  • 悪い例
= link_to user_path(user), class: 'text-decoration-none' do
  = user.name
  • 良い例
= link_to user.name, user_path(user), class: 'text-decoration-none'  

理由

  • リンク内がテキストだけであるというケースは、1行で書いた方が見やすい。リーダブルである
  • なるべくインデントない方が見やすい

パーシャルでインスタンス変数を使用しない

PRのタイトル、コメントは適切か?

  • そのPRで自分がどんな実装をしたのか?
  • なぜそのコードを記述したのか?

などを正確に記述できているか?

1
0
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
1
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?