[E-1] 2-04.Add Prompt/Personal Prompts for LangChain #69
Conversation
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
[Title] Add Prompt/Personal Prompts for LangChain [Version] initial [Language] ENG [Packages] langsmith, langchain, langchainhub - LangChain Hub에 프롬프트 Push / Pull - 종류별 Prompt 예제
[Review] 1. Review OS: Windows 2. Template Rule 가이드를 준수 하였는가: No 3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: YES 4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES 5. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES 6. 모든 코드가 동작에 오류 없이 동작하는가? : No 7. 기타 의견: - 템플릿가이드에 맞게 BestPractice, Key Factors 등을 삭제 부탁 드리며, 목차는 불렛(- )으로 변경, 목차항목은 ## 로 통일 부탁 드립니다. 일부 수정하였으니 참고하여 작업 부탁 드립니다.
r14minji
requested changes
Jan 2, 2025
There was a problem hiding this comment.
[Review]
- Review OS: Mac
- Template Rule 가이드를 준수 하였는가: No
- Heading1(#)은 제목만 단 한 번 사용하기로 했습니다.
- 해당 가이드 규칙 문서 공유드립니다.
- Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: YES
- 앵커가 동작하기는 하는데요. Heading2(##) 기준으로 앵커를 작성하는 일관된 제목 규칙을 가져가는 것에 대해서는 어떻게 생각하시나요?
- 위에 PM님이 작성해주신 양식과 동일하게 수정해주시면 감사하겠습니다.
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
- 모든 코드가 동작에 오류 없이 동작하는가? : No (더 확인해보고 자세히 작성해둘게요)
- 기타 의견:
jeong-wooseok
requested changes
Jan 2, 2025
There was a problem hiding this comment.
[Review]
Review OS: Windows
Template Rule 가이드를 준수 하였는가: No
99폴더의 표준템플렛의 양식에 맞춰서 ##을 목차이름으로 맞춰주세요.
keyFactors, BestPractice 등의 문구 및 서론이 길어서 타콘텐츠와 균형이 맞지않아서, 요약 및 표준화 부탁드려요
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: No
목차이름을 불렛 (- )으로 통일해주시고 링크 재확인해주세요
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
모든 코드가 동작에 오류 없이 동작하는가? : No
뒷편에 코드가 오류 발생합니다. 확인 부탁 드려요
기타 의견:
r14minji
previously approved these changes
Jan 4, 2025
jeong-wooseok
approved these changes
Jan 4, 2025
|
[Review]
|
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.
[Title] Add Prompt/Personal Prompts for LangChain
[Version] initial
[Language] ENG
[Packages] langsmith, langchain, langchainhub
코드 리뷰 부탁드립니다. 감사합니다.