Feature/105 add setup clickhouse prompt dir#106
Conversation
setup.py
Outdated
| import os | ||
| import glob | ||
|
|
||
| with open("docs/README.md", "r", encoding="utf-8") as fh: |
There was a problem hiding this comment.
❗ docs/README.md 에 프로젝트 실행관련을 추가해두어서 해당 부분을 의도한 바이기는 합니다.
There was a problem hiding this comment.
❓
.env.example,pyproject.toml은 어떤 이유로 삭제된지 알 수 있을까요?
.env.example -> 습관적으로 삭제했네요...특별한 이유는 없습니다.
pyproject.toml 삭제하고 install하면서 다시 복구가 안된것같습니다..ㅎㅎ
❗ black 으로 formatting이 필요한것으로 보입니다
네 formatting 하겠습니다.
❗ docs/README.md 에 프로젝트 실행관련을 추가해두어서 해당 부분을 의도한 바이기는 합니다.
감사합니다.
setup.py
Outdated
| long_description=long_description, | ||
| long_description_content_type="text/markdown", | ||
| packages=find_packages(), # my_package를 자동으로 찾음 | ||
| packages=find_packages() + ['prompt'], # prompt 패키지 직접 추가 |
There was a problem hiding this comment.
👍 오 md 파일이 일반적으로는 포함이 안되는게 이 이유에서였군요!!
❓ 아래 추가로만으로도 빌드파일에 md파일이 포함되는것으로 보입니다. 혹시 data_files와 include_package_data 는 어떤 이유로 추가된건지 알수있을까요!?
packages=find_packages() + ["prompt"], # my_package를 자동으로 찾음
package_data={
"prompt": ["*.md", "*.py"], # prompt 디렉토리의 모든 .md 파일 포함
},
There was a problem hiding this comment.
👍 오 md 파일이 일반적으로는 포함이 안되는게 이 이유에서였군요!!
저만 안돼는게 아니였군요 ㅎㅎ
❓ 아래 추가로만으로도 빌드파일에 md파일이 포함되는것으로 보입니다. 혹시 data_files와 include_package_data 는 어떤 이유로 추가된건지 알수있을까요!?
packages=find_packages() + ["prompt"], # my_package를 자동으로 찾음 package_data={ "prompt": ["*.md", "*.py"], # prompt 디렉토리의 모든 .md 파일 포함 },
패키지를 사용하는 모든 사용자를 고려해 오픈소스에 적합하다고 생각되어 포함하였습니다.
There was a problem hiding this comment.
👍 @bongkyunSON 추가 리뷰가 너무 늦었습니다ㅜㅜ!! 수정내역은 컨플릭트 해결 및 중복부분 삭제입니다.
중복되던 내용
packages=find_packages(), # my_package를 자동으로 찾음
packages=find_packages() + ["prompt"], # prompt 패키지 직접 추가
# 프롬프트 파일들을 찾습니다
prompt_files = glob.glob("prompt/*.md")
prompt_files = [os.path.basename(f) for f in prompt_files]
package_data={
"prompt": ["*.md"],
},
include_package_data=True -> False
- 이 옵션은 True로 주게되면 MANIFEST 파일을 참고한다고 하네요!
수정사항 확인해주시면 머지해도 될듯합니다!
#️⃣ Issue Number
📝 요약(Summary)
💬 To Reviewers (선택)
PR Checklist
reference) How to Code Review