Skip to content

リファクタリング - #39

Merged
dev-satoshi merged 1 commit into
mainfrom
refactoring
Aug 15, 2026
Merged

dev-satoshi merged 1 commit into
mainfrom
refactoring

Conversation

@dev-satoshi

@dev-satoshi dev-satoshi commented May 16, 2025 •

Copy link
Copy Markdown
Owner

やったこと

  • setupスクリプトを scripts/setup 配下に移動
  • 移動に伴う参照パスを更新

変更内容

scripts/ 直下にあったセットアップ用スクリプト 7 本を scripts/setup/ に移動しました。既存の scripts/update/ と同じ構成になります。

参照側も合わせて更新しています。

  • install.sh → scripts/setup/setup.sh を呼ぶ
  • scripts/setup/setup.sh → 各スクリプトを scripts/setup/ から呼ぶ

動作確認

CI と同じ ShellCheck をローカルで実行し、エラーがないことを確認しました。

$ find . -type f -name '*.sh' -not -path './.github/*' -print0 | xargs -0 shellcheck --shell=bash
$ echo $?
0

あわせて以下も確認しています。

  • 全スクリプトの zsh -n による構文チェック
  • install.sh と scripts/setup/setup.sh が呼ぶパスが全て実在すること
  • 旧パス scripts/*.sh への参照が残っていないこと

🤖 Generated with Claude Code

@dev-satoshi dev-satoshi self-assigned this May 16, 2025
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@dev-satoshi
dev-satoshi merged commit ab7603b into main Aug 15, 2026
1 check passed
@dev-satoshi

Copy link
Copy Markdown
Owner Author

最新の main(#40 マージ済み)を基点にブランチを作り直し、force-push しました。

以前の版からの変更点は次の 3 点です。

  1. コピーではなく移動にした — 旧 scripts/*.sh が削除されていなかったため、スクリプトが 2 箇所に重複していました。git mv で移動し直しています。
  2. 内容が最新になった — 以前の版は 81dbaf8 より前の main が基点だったため、追加される scripts/setup/ 配下が古い内容でした(android-34 のまま、.zprofile のリンク欠落など)。install.sh の参照先が新パスに変わる関係で、実行されるのは古い方になっていました。
  3. README のタイポを持ち込まなくなった — ## Installation が ## Instal になっていた変更が含まれていました。今回は README に変更はありません。

@dev-satoshi
dev-satoshi deleted the refactoring branch August 15, 2026 10:31
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