Skip to content

start chore/cleanup-rails7-code-comments#9710

Draft
shibainurou wants to merge 1 commit intomainfrom
chore/cleanup-rails7-code-comments
Draft

start chore/cleanup-rails7-code-comments#9710
shibainurou wants to merge 1 commit intomainfrom
chore/cleanup-rails7-code-comments

Conversation

@shibainurou
Copy link
Copy Markdown
Contributor

@shibainurou shibainurou commented Mar 1, 2026

Issue

概要

Rails 7にアップグレード後に対応を予定していたTODOコメント3つの対応を実施する。

詳細

  1. # Pull Request #4182(https://github.com/fjordllc/bootcamp/pull/4182) で Rails 7 への移行完了後に削除する

対応:下記にコードコメントを消すだけで問題無しとコメントがあるので、コメント削除のみ。
#9324 (comment)

  1. # TODO: Rails 7に更新後、 `ComparisonValidator` を使うように直す。
    # refs: https://github.com/rails/rails/pull/40095
    # validates :end_at, greater_than: :start_at

対応:日付の比較にComparisonValidatorを使用する。

参考リンク
#4030 (comment)

  1. bootcamp/Gemfile

    Lines 50 to 52 in 8de3e3d

    gem 'net-imap', require: false
    gem 'net-pop', require: false
    gem 'net-smtp', require: false # TODO: Remove it if you use rails 7.0.1

対応:Ruby 3.1でnet-imap, net-pop, net-smtp, が削除されたので個別に記述していたが、Rails側で対応するようになったのでrequireを削除する。

参考リンク
rails - Commit 48f2f1a
Temporarily add net-gems as dependencies of frameworks that use mail

変更確認方法

  1. {branch_name}をローカルに取り込む

Screenshot

変更前

変更後

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 1, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/cleanup-rails7-code-comments

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant