Skip to content

Conversation

@ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • Lang2SQL 프로젝트의 LLM 관련 모듈 구조를 개선하여 llm_utils/ 디렉토리를 utils/llm/으로 이동
  • 프로젝트 전체의 모듈 구조 일관성을 확보하고 유지보수성을 향상시키기 위한 리팩토링 작업
  • 모든 관련 파일의 import 경로를 utils.llm.*로 변경하여 기존 기능은 유지하면서 구조만 개선

💬 To Reviewers (선택)

  • 모든 import 경로가 올바르게 수정되었는지 확인 부탁드립니다
  • 기존 기능이 정상적으로 작동하는지, 특히 engine/query_executor.pyinterface/ 모듈들의 동작을 중점적으로 검토해주세요
  • 테스트 코드의 import 경로 변경이 올바른지 확인해주세요

PR Checklist

  • 모든 import 경로가 올바르게 수정되었는지 확인
  • 기존 기능이 정상적으로 작동하는지 테스트
  • 테스트 코드가 새로운 경로에서 정상 실행되는지 확인
  • Streamlit 인터페이스가 정상적으로 동작하는지 확인

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 좋습니다아아! 조만간 docs 업데이트도 제가 한번 하겠습니다.

@ParkGyeongTae ParkGyeongTae merged commit 472c892 into master Oct 7, 2025
2 checks passed
@ParkGyeongTae ParkGyeongTae self-assigned this Oct 7, 2025
@ehddnr301 ehddnr301 deleted the 190-move-llm-utils-to-utils branch October 11, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants