Skip to content

Conversation

@ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • 신규 추가된 LLMResponseParse 모듈의 테스트코드 추가

💬 To Reviewers (선택)

  • 없음

PR Checklist

reference) How to Code Review

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

@ParkGyeongTae
Copy link
Contributor Author

image

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.

👍 Lang2SQL의 첫 테스트 코드군요!

❓pytest는 라이브러리의 필수적인 부분이 아닌것 같아 setup.py에 추가해야하는지 잘 모르겠는데 혹시 requirements.txt에만 추가하면 될까요!?

image

@ParkGyeongTae
Copy link
Contributor Author

ParkGyeongTae commented May 6, 2025

@ehddnr301 지금.. pytest를 명시해서 설치하지 않아도 다른 패키지 설치 중 의존성때문에 pytest가 같이 깔리는 것 같아요!
우선 이 PR에 requirements.txt와 setup.py에 pytest를 명시해서 설치하는 부분 추가하겠습니당!

@ParkGyeongTae
Copy link
Contributor Author

@ehddnr301 추가 했습니당~

@ParkGyeongTae ParkGyeongTae merged commit 4333383 into master May 18, 2025
1 check passed
@ehddnr301 ehddnr301 deleted the feature/92-add-test-code-llm-response-parser branch September 7, 2025 07:07
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.

3 participants