Skip to content

バージョンバンプスクリプトのiOS設定検出を修正#5509

Merged
TinyKitten merged 1 commit intodevfrom
fix/bump-version-ios-detection
Mar 1, 2026
Merged

バージョンバンプスクリプトのiOS設定検出を修正#5509
TinyKitten merged 1 commit intodevfrom
fix/bump-version-ios-detection

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Mar 1, 2026

Summary

  • scripts/bump-version.js の iOS 設定ブロック検出を includes('ios: {') から正規表現 /^ ios:\s*\{/ に変更し、expo-build-properties プラグイン内の ios: と誤マッチしないようにした
  • app.config.ts から expo-build-propertiesios ブロック内に誤挿入されていた buildNumber を削除

Test plan

  • npm run typecheck が通ること
  • canary リリースでバージョンバンプが正しく動作すること

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

  • Chores
    • ビルド設定を最適化し、iOS ビルドナンバー設定の重複を削除しました。
    • バージョンバンプスクリプトの iOS セクション検出ロジックをより堅牢に改善しました。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c7c5953 and ddfd3da.

📒 Files selected for processing (2)
  • app.config.ts
  • scripts/bump-version.js
💤 Files with no reviewable changes (1)
  • app.config.ts

📝 Walkthrough

Walkthrough

iOS のビルド番号設定を簡潔にするため、app.config.ts 内の expo-build-properties プラグイン設定からプラグインレベルの buildNumber エントリを削除し、bump-version.js の iOS セクション検出を正規表現で厳密化しました。

Changes

Cohort / File(s) Summary
iOS ビルド設定の整理
app.config.ts
expo-build-properties プラグイン設定内のプラグインレベル iOS buildNumber エントリ(値: 2509)を削除。トップレベルの ios.buildNumber は維持。末尾の空行も削除。
バージョン検出の厳密化
scripts/bump-version.js
iOS セクションの検出条件を substr チェック(lines[i].includes('ios: {'))から正規表現パターン(^ ios:\s*\{)に変更し、境界検出をより正確に。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

ios, ci/cd

Poem

🐰 プラグインレベルの buildNumber よ、さようなら
正規表現が iOS をキャッチしテキパキと
シンプルに、クリアに、ビルドは進む
余計なものは削ぎ落として、本質だけ
バージョン管理、今日も前へ ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed プルリクエストのタイトルは、iOS設定検出の修正という変更の主要な目的を正確に反映しており、変更セット全体の中心的な改善を明確に要約しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/bump-version-ios-detection

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

@TinyKitten TinyKitten self-assigned this Mar 1, 2026
@TinyKitten TinyKitten merged commit 86a5658 into dev Mar 1, 2026
5 checks passed
@TinyKitten TinyKitten deleted the fix/bump-version-ios-detection branch March 1, 2026 06:07
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