Repository navigation
canaryをdevに取り込み、#7146の競合を解消 - #7147
Merged
Merged
Conversation
Bump version for canary release (#6565)
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e/ride-review-screen-walkthrough # Conflicts: # assets/translations/en.json # assets/translations/ja.json # src/constants/storage.ts
…e/ride-review-prefectures # Conflicts: # src/screens/RideReview.test.tsx # src/screens/RideReview.tsx
振り返りに訪れた都道府県と都道府県ごとの乗車距離を表示し、乗車ログに駅の都道府県を記録するようにした
chore: Bump version for PR #7131
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
振り返り画面の注記から、直線距離で代える区間についての一文を外した
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…lete 保存済みの乗車の記録が無いときは、設定の「記録をすべて削除」を押せないようにした
…-dev # Conflicts: # src/screens/AppSettings.tsx
chore: Bump version for PR #7139
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013GtxnGNUDKFjG4DyAk3ATs
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
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.
概要
canary に振り返り機能のブランチが直接マージされ、dev には同じ機能が #7145 の1コミットで入っています。履歴が別々になり、dev → canary の #7146 が競合していました。canary を dev に取り込んで、競合を解消します。
変更の種類
変更内容
競合した8ファイルは、すべて dev 側の内容を残しました。canary 側は、dev だけにある新しい変更が入る前の形だったためです。
build_android_canary.yml/build_android_production.yml/docs/android-build-workflows.md: Maps キーをDEV_GOOGLE_MAPS_API_KEYとPRODUCTION_GOOGLE_MAPS_API_KEYに分ける変更docs/ride-log.md/src/constants/storage.ts/src/store/atoms/rideLog.ts/src/screens/RideLogSettings.tsx/src/screens/RideLogSettings.test.tsx: 設定の「振り返り」を開いたかを記録する既読フラグ(RIDE_LOG_SETTINGS_SEEN)とそのテスト競合していない canary 側の変更(ウォークスルー、都道府県表示、ビルド番号の bump など)は、そのまま dev に入ります。
このPRは squash ではなく、マージコミットでマージしてください。 squash すると canary の履歴が dev に入らず、#7146 の競合が残ります。
テスト
npm run lintが通ることnpm testが通ることnpm run typecheckが通ることローカルで実行しました。
npm testは319スイート・3565テストがすべて通りました。関連Issue
Refs #7146
スクリーンショット(任意)
UI変更なし: 既存の2つのブランチを統合して競合を解消するだけで、新しい画面の変更はありません。
🤖 Generated with Claude Code
https://claude.ai/code/session_013GtxnGNUDKFjG4DyAk3ATs
Generated by Claude Code