Repository navigation
[CHORE/#41,#42] CI/CD 의존성 설정 변경 - uv 의존성 사용, AWS Secret Rotation 대응 코드 적용 - #42
Merged
Merged
Conversation
[FEAT#38] MVP 버전 배포
…tion 점수 모두 고려하여 예외 처리 추가
…_PATH, AMPLITUDE_API_KEY, LOGFIRE_TOKEN 환경 변수 추가
…client 값을 웹 클라이언트로 변경
…ript-api 추가, prod_deploy.yml에 YOUTUBE_API_KEY 추가, agent.py에서 YoutubeTranscriber 추가 및 yt_dlp_downloader.py에서 비디오 다운로드 관련 코드 정리
…tubeTranscriber에서 로그 출력 개선
This was
linked to
issues
Sep 8, 2025
Closed
juhy0987
changed the title
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the project's dependency management from requirements.txt to uv, and transitions CI/CD from AWS ECS to Google Cloud Run. Key infrastructure and tooling changes are implemented to modernize the deployment pipeline.
- Replaces pip with uv for dependency management and removes requirements.txt files
- Updates GitHub Actions workflows to use uv and deploy to Google Cloud Run instead of AWS ECS
- Adds new YouTube transcription functionality and modifies processing pipeline
- Updates database connection handling to support AWS Secrets Manager with connection pooling
Reviewed Changes
Copilot reviewed 19 out of 21 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | New dependency configuration file replacing requirements.txt |
| requirements-dev.txt | Removed in favor of uv dev-dependencies |
| .github/workflows/*.yml | Updated CI/CD workflows to use uv and Google Cloud Run |
| Dockerfile | Modified to use uv instead of pip for package installation |
| src/common/rdb/common/database.py | Enhanced database connection with AWS Secrets Manager integration |
| src/common/processor/transcriber/strategies/youtube_transcriber.py | New YouTube transcription strategy implementation |
| src/common/processor/agent.py | Modified processing pipeline to use YouTube transcriber |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
This branch had an error being deployed
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.
연관 이슈
구현 내용
TODO
논의 사항