-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/(CI)GitHub ActionsによるCIワークフローの追加 #8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
fix(.gitignore): Pythonのキャッシュファイルの無視設定を修正 refactor(main.py): FastAPIのライフスパン管理を追加
forwardOne
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
チケット駆動のためIssueの利用、テストコードとGithub Actionsのワークフロー追加
| run: pip install -r requirements.txt | ||
| - name: Run tests | ||
| env: | ||
| GEMINI_API_KEY: ${{ secrets.GEMINI_API_KEY }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
secrets を使ってAPIキーを渡すことで、CIでの認証エラーを解決。
| # 以下の"YOUR_API_KEY_HERE"を実際のAPIキーに置き換えてください。 | ||
|
|
||
| # GEMINI_API_KEY="YOUR_API_KEY_HERE" | ||
| # ファイル名を.envに変更して使用してください。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ユーザー向けに環境変数のテンプレートを追加。
| // @testing-library/jest-dom の拡張アサーションを有効化 | ||
| import '@testing-library/jest-dom'; | ||
|
|
||
| // Mock for window.matchMedia used in use-mobile hook |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ブレークポイント対応
GitHub ActionsによるCIワークフローを追加します。
テストコードは試験的に実装しました。
このワークフローは、
mainブランチへのプッシュまたはプルリクエスト時に、バックエンド(Python/pytest)とフロントエンド(Node.js/Vitest)のテストを自動実行します。Closes #7