Closed maixhashi closed 2 months ago
このプルリクエストにて何をしたのか? Characterモデルレコードの抽出条件を追加
このプルリクエストにて何をしたのか?
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要) なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい Characterレコードとして近いものが抽出されるようにする。
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)
なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい
どの環境でどんな動作チェックをしたか 動作確認をした事についてスクショなどがあるとわかりやすくて良い 別途テスト作成予定
どの環境でどんな動作チェックをしたか
動作確認をした事についてスクショなどがあるとわかりやすくて良い
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報 国の名前や、地名はCharacterモデルレコードとして保存しないようにするための条件の実装で、 gem 'active_json' で国名のリストを作りCharacterとしてを使わないこととした。
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報
gem 'active_json'
Natto::MeCabで国の名前としての分類フラグが実装されているため。
Natto::MeCab
GitHub Issue Ticket
やった事
このプルリクエストにて何をしたのか?
Characterモデルレコードの抽出条件を追加なぜやるのか
GitHub Issue で説明できない捕捉的な事項 (GitHub Issue の説明で十分であればここは不要)
なぜこのプルリクエストが必要と考えたかについて説明があるとレビュワーがわかりやすい
Characterレコードとして近いものが抽出されるようにする。動作確認
どの環境でどんな動作チェックをしたか
動作確認をした事についてスクショなどがあるとわかりやすくて良い
別途テスト作成予定Refs (レビューにあたって参考にすべき情報)(Optional)
関連するプルリクエストやイシュー、コンフルリンクなど、レビュワーがレビューするにあたっての補足情報
国の名前や、地名はCharacterモデルレコードとして保存しないようにするための条件の実装で、gem 'active_json'
で国名のリストを作りCharacterとしてを使わないこととした。Natto::MeCab
で国の名前としての分類フラグが実装されているため。