リファクタリング - #39
Merged
Merged
リファクタリング#39
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
dev-satoshi
force-pushed
the
refactoring
branch
from
August 15, 2026 10:31
b1f6e92 to
ee5cbce
Compare
Owner
Author
|
最新の main(#40 マージ済み)を基点にブランチを作り直し、force-push しました。 以前の版からの変更点は次の 3 点です。
|
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.
やったこと
scripts/setup配下に移動変更内容
scripts/直下にあったセットアップ用スクリプト 7 本をscripts/setup/に移動しました。既存のscripts/update/と同じ構成になります。参照側も合わせて更新しています。
install.sh→scripts/setup/setup.shを呼ぶscripts/setup/setup.sh→ 各スクリプトをscripts/setup/から呼ぶ動作確認
CI と同じ ShellCheck をローカルで実行し、エラーがないことを確認しました。
あわせて以下も確認しています。
zsh -nによる構文チェックinstall.shとscripts/setup/setup.shが呼ぶパスが全て実在することscripts/*.shへの参照が残っていないこと🤖 Generated with Claude Code