Skip to content

Simplify Makefile commands for local development#23

Closed
suna-big wants to merge 15 commits intomainfrom
simplify-makefile-commands
Closed

Simplify Makefile commands for local development#23
suna-big wants to merge 15 commits intomainfrom
simplify-makefile-commands

Conversation

@suna-big
Copy link
Copy Markdown
Collaborator

@suna-big suna-big commented Dec 10, 2025 •
edited
Loading

Summary

  • Makefile のコマンドを run-local に統一し、local 環境のみをサポート
  • cloud9 環境の設定を削除
  • Docker Compose で MySQL と Redis を簡単にセットアップできるように改善
  • デフォルト環境を local に設定

変更内容

設定の簡素化

  • Makefile: run-local ターゲットで ENV_ENVIRONMENT=local を設定
  • config.go: cloud9 ケースを削除し、local ケースのみをサポート
  • app.go: Environment のデフォルト値を local に変更

Docker 対応

  • docker-compose.yml: MySQL 8.0 と Redis 7 の設定を追加
    • DDL の自動実行によるスキーマ初期化
    • ヘルスチェック設定
    • データ永続化

ドキュメント

  • README.md: Docker を使用した実行方法を追加

実行方法

# MySQL と Redis を起動
$ docker-compose up -d

# アプリケーションを起動
$ make run-local

# 停止する場合
$ docker-compose down

Test plan

  • make run-local コマンドが local 環境で正常に動作することを確認
  • Docker Compose で MySQL と Redis が起動することを確認
  • アプリケーションが MySQL と Redis に正常に接続できることを確認
  • ヘルスチェックエンドポイント（/）が HTTP 200 を返すことを確認

🤖 Generated with Claude Code

@suna-big suna-big force-pushed the simplify-makefile-commands branch from 06be92b to d93c106 Compare December 10, 2025 10:53
@suna-big @claude
Simplify Makefile commands and clarify Cloud9 requirement
4233ebf 
- Remove `run-local` target from Makefile (local execution not supported)
- Rename `run-cloud9` to `run` for simpler command
- Add notice in README that project is designed for AWS Cloud9
- Clarify that MySQL and Redis do not start automatically in local environment

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@suna-big suna-big force-pushed the simplify-makefile-commands branch from d93c106 to 4233ebf Compare December 10, 2025 10:57
suna-big and others added 6 commits December 13, 2025 13:36
@suna-big
test
d5aa53b
@suna-big
fix
72385b7
@suna-big
fix: bug
649b16b
@suna-big
fix
fca8ebd
@suna-big
fix
981a3b3
@suna-big @claude
refactor: improve code clarity and add debugging support
f973862 
- Simplify comments in load test configuration for better readability
- Add pprof endpoint for performance profiling
- Update xray segment namer to suna-big

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@suna-big suna-big changed the title Simplify Makefile commands and clarify Cloud9 requirement Simplify Makefile commands for local development Dec 24, 2025
suna-big and others added 8 commits December 24, 2025 10:30
@suna-big @claude
fix: change environment from ec2 to local
7c87b2b 
- Update ENV_ENVIRONMENT from ec2 to local in Makefile
- Align with PR #23 to support local development only

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@suna-big @claude
refactor: rename run target to run-local
9eaa32e 
- Rename 'run' to 'run-local' for clarity
- Make it explicit that this runs in local environment

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@suna-big @claude
refactor: replace ec2 environment with local
d104176 
- Change environment case from "ec2" to "local" in config
- Align with local-only development approach

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@suna-big @claude
refactor: update local environment configuration
51105bc 
- Update README.md to use 'make run-local' command
- Change local config to use RawDBConfig instead of SecretsManager
- Set local database connection to localhost with default settings

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@suna-big @claude
revert: remove unnecessary service.go changes
ebd5a8e 
- Revert pprof endpoint addition
- Revert xray segment namer change
- Keep only local environment configuration changes

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@suna-big @claude
fix: change default environment to local
8fe287f 
- Update default Environment value from ec2 to local
- Align with local-only development approach

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@suna-big @claude
feat: add Docker Compose for MySQL and Redis
a71d35e 
- Add docker-compose.yml for easy local development
- Configure MySQL 8.0 with automatic schema initialization
- Configure Redis 7 with persistence
- Update README.md with Docker usage instructions

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@suna-big @claude
feat: add test data insertion script
475c67d 
- Add scripts/insert-test-data.sh for easy test data setup
- Insert sample data: 5 genres, 5 series, 3 seasons, 3 episodes
- Update README.md with test data insertion instructions

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@suna-big suna-big closed this Dec 24, 2025
@suna-big
Copy link
Copy Markdown
Collaborator Author

suna-big commented Dec 24, 2025

移設したので不要になりました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@suna-big