不要なファイルの削除と設定ファイルの整理 #1031
不要なファイルの削除と設定ファイルの整理 #1031
Conversation
- golangci.ymlをapi/配下に移動 - my.cnfをmysql/配下に移動 - 使用していないmysql/prdDb/配下のSQLファイルを削除 - 使用していないmysql/feature/配下のSQLファイルを削除 - 不要なfinанsu-ER.drawio、test.ymlを削除 - golangci-lint.ymlとcompose.db.ymlのパスを修正 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Summary of ChangesHello @Kubosaka, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! このプルリクエストは、プロジェクトの全体的な健全性と保守性を向上させるためのリファクタリング作業です。不要なファイルを削除し、設定ファイルを論理的な場所に移動することで、コードベースをクリーンアップし、開発環境の混乱を減らすことを目的としています。これにより、将来の開発作業がより効率的になることが期待されます。 Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Pull request overview
This PR removes unused production database initialization files and reorganizes configuration files to improve project structure and maintainability.
Changes:
- Removed 38 unused SQL files from
mysql/prdDb/containing production database seed data - Removed 2 unused SQL files from
mysql/feature/ - Deleted unused
test.ymlandfinansu-ER.drawiofiles - Moved configuration files to appropriate directories (
golangci.yml→api/.golangci.yml,my.cnf→mysql/my.cnf) - Updated paths in
compose.db.ymland.github/workflows/golangci-lint.ymlto reference relocated configuration files
Reviewed changes
Copilot reviewed 41 out of 44 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| test.yml | Removed unused test configuration file |
| mysql/prdDb/*.sql | Removed 38 production database seed SQL files (tables, initial data) |
| mysql/feature/*.sql | Removed 2 unused feature development SQL files |
| compose.db.yml | Updated my.cnf path to mysql/my.cnf |
| .github/workflows/golangci-lint.yml | Updated golangci-lint config path to api/.golangci.yml |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
このプルリクエストは、不要なファイルを削除し、設定ファイルを整理する素晴らしいリファクタリングですね。特に、mysql/prdDb/にあった本番用の初期データをリポジトリから削除したことで、セキュリティが向上し、プロジェクトがよりクリーンになった点を高く評価します。
しかし、1点だけ重大な懸念事項があります。mysql/prdDb/ディレクトリを削除したことにより、compose.prod.db.ymlファイル内の本番データベースサービス(finansu-prd-db)のボリューム設定が壊れてしまいます。具体的には、以下の行です。
volumes:
- ./mysql/prdDb:/docker-entrypoint-initdb.d # 初期データこのままでは、本番環境のデプロイ時にデータベースの初期化に失敗する可能性が非常に高いです。本番環境の初期データは別の方法で管理されるようになったかと思いますので、compose.prod.db.ymlからこのボリュームマウントを削除するか、新しいデータソースを指すように修正する必要があります。この修正を本プルリクエストに含めることを強く推奨します。
Deploying finansu with
|
| Latest commit: |
9be4311
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://5463ac33.finansu.pages.dev |
| Branch Preview URL: | https://delete-remove-nouse-files.finansu.pages.dev |
|
@TkymHrt |
対応Issue
なし（リファクタリング作業）
概要
プロジェクト内の不要なファイルを削除し、設定ファイルを適切なディレクトリに移動しました。
主な変更内容
golangci.ymlをapi/.golangci.ymlに移動my.cnfをmysql/my.cnfに移動mysql/prdDb/配下のSQLファイル（38ファイル）を削除mysql/feature/配下のSQLファイル（2ファイル）を削除finansu-ER.drawio、test.ymlを削除.github/workflows/golangci-lint.ymlのパスを修正compose.db.ymlの my.cnf パスを修正削除理由
mysql/prdDb/配下のSQLファイルは本番データベースの初期データで、開発環境では使用していないmysql/feature/配下のSQLファイルは古い機能開発時の一時的なファイルfinansu-ER.drawioは ER図ファイルだが、最新のスキーマと同期されていないtest.ymlは使用されていない設定ファイル画面スクリーンショット等
なし（バックエンドのファイル整理のため画面変更なし）
テスト項目
make buildでビルドが正常に完了することmake run-dbでデータベースが正常に起動することmake runでアプリケーションが正常に起動することapi/.golangci.ymlのパスで）備考
🤖 Generated with Claude Code