Skip to content

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

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

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 추가 했습니당~

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