-
Notifications
You must be signed in to change notification settings - Fork 75
Release 2026-01-14 09:56:43 +0000 #9497
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
github-actions
wants to merge
57
commits into
production
Choose a base branch
from
main
base: production
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+4,324
−9,747
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
前回のビュー変更(4003c4e2b)に合わせて、コントローラーの権限 チェックを修正し、関連するテストを追加した。
変更内容: - answer.user.adviser から answer.user.adviser? に変更(29行目) - メソッド呼び出し(?付き)に統一し、_comment.html.slimと同じパターンに この修正により、両ファイルで会社ロゴ表示の条件判定が統一されます。
- _answer.html.slim: traineeのチェックを削除しadviser?のみに - _comment.html.slim: モバイル表示でもadviser?を使用するよう修正
- rails app:update で生成されたファイルを更新 - 新しいフレームワークデフォルト設定を追加 - 環境設定ファイルを更新 - 新しいエラーページとアイコンを追加 - bin/ci, bin/dev などの新しいスクリプトを追加
- AI::AnswerGenerator → Ai::AnswerGenerator - AIAnswerCreateJob → AiAnswerCreateJob - AIAnswerCreator → AiAnswerCreator - inflections.rbからAIアクロニムを削除
- image_processing: >= 1.12.2 に更新 (CVE-2022-24720対応) - react-rails: 特定コミットにピン留め (mainブランチの不安定さを回避) - benchmark gem追加 (Ruby 3.5対応) - rubocop関連gemをdevelopment,testグループに移動
CircleCI: - libheif-devを追加してHEICサポートを有効化 - importmap auditをコメントアウト (将来のimportmap移行用) CloudBuild: - DeleteDBステップのリトライ設定を強化 (12回/10秒間隔)
- 認証が必要なページへのvisitをvisit_with_authに変更 - ActionMailer::Base.deliveriesをsetupでクリア - findをassert_selectorに変更してアサーションを追加 - assert_differenceでDB変更を検証 - fill_optionsの一貫性を確保 - HEICテストのskipを削除 (CIでlibheif対応済み)
- Discord: ActiveSupport::Configurableをclass_attributeに変更 - ReportExporter: Zip::File.open APIの変更に対応 - User: Vips::Errorをrescue対象に追加
- solid_queueテーブル作成: table_exists?で存在チェック - good_jobテーブル削除: table_exists?で存在チェック CI(schema.rb読み込み後)と本番環境(db:migrate)の 両方で正常に動作するように修正。
has_css?/has_no_css?は真偽値を返すだけでアサーションではないため、 assert_selector/assert_no_selectorに修正。
page-tabs → .page-tabs(CSSクラスセレクタの正しい形式)
good_jobをsolid_queueに移行
Rails 7.2.2.2 → 8.1.1 アップグレード
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
- yarn installをnpm installに変更 - yarn.lockをpackage-lock.jsonに変更 - yarnのインストール処理を削除
Dockerfileをyarnからnpmに移行
yarnを使わずにnpmでビルドするため、 SHAKAPACKER_NODE_MODULES_BIN_PATH環境変数を設定。
ShakapackerのNode modules bin pathを設定
bin/yarn lint → npm run lint
yarnからnpmへの移行に伴い、shakapackerが node_modules/.binを見つけられるように環境変数を設定。
shakapacker v9.xではwebpack_loaderは非推奨となり、 javascript_transpilerが使用される。デフォルトがswcになったため、 明示的にbabelを指定。
境界ギリギリの日付(13日/14日)ではタイミングで テストが不安定になるため、明確なマージンに変更: - jobseeker: 1週間前(delayedに含まれない) - muryou: 3週間前(delayedに含まれる)
- npm: ^9.5.0 → 7.2.3 (gemと同じバージョンに) - config: javascript_transpiler → webpack_loader (v7.x用に戻す)
- gem: 7.0 → 9.0 - npm: 7.2.3 → 9.5.0 - config: webpack_loader → javascript_transpiler (v9の新設定名) CSS Modulesは使用していないため、破壊的変更の影響なし。
bin/lintをyarnからnpmに変更
マイグレーションファイルと整合しない方のインデックスを削除
…pansion-in-lint Prettier実行時のglob展開が環境に依存しないようにする
コメントの不要な余白を削除
…-user-index commentsテーブルの重複したインデックスを削除
メール送信エラー(Postmark::InactiveRecipientError等)が 発生してもサイト内通知が確実に作成されるように、 実行順序をサイト内通知→メール通知に変更。
…order お知らせ通知の順序を変更: サイト内通知を先に実行
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.