Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

cjLee-cmd
Copy link
Contributor

[Team] Existing Content Development Team 1
[Title] 03-LangChain Hub
[Version] initial
[Language] ENG
[Packages] langchain_core.prompts, langchain_openai
LangChain PromptTemplate Tutorial

@teddylee777 teddylee777 requested review from musangk and fastjw January 4, 2025 14:48
Copy link
Contributor

@musangk musangk left a comment

Choose a reason for hiding this comment

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

[Review Checklist]

  • Review OS:

    • Linux
  • Template Rule 준수 여부:

    • Template Rule 가이드를 준수하였는가? (Check 표기)

    Colab과 Langchain Academy Badge가 보이지 않는 문제 수정 : ced0423

  • Table of Contents 링크 확인:

    • Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)

    링크가 적용되지 않은 문제 수정 : b410c9f

  • 이미지 파일명 검토:

    • 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)
  • 최신 import 방식 사용 여부:

    • import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)

    최신 import 구문 적용 수정 : f909705

  • 코드 동작 확인:

    • 모든 코드가 오류 없이 동작하는가? (Check 표기)
    • Warning 발생 시 코멘트에 적어주세요.
  • 제출 파일 확인:

    • 꼭 필요한 파일들만 제출되었는가?
      • 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)
  • 기타 의견:

    • 작성해주신 내용을 리뷰하면서 Langchain Hub에 대한 내용을 파악 할 수 있어서 좋았습니다.
    • 이번 PR은 수정이 1줄이지만, 제목이 해당 파트 전체를 의미하여서 해당 파일 전체를 리뷰하였습니다. ( 관련 PR : #101 )
    • 사소한 수정 사항은 직접 수정하였고, 각 항목아래에 설명과 함께 링크를 달아 두었으니 참고하여 주시기 바랍니다. 관련하여 질문사항이나 의도와 다르게 수정이 된 부분이 있다면 알려주시면 수정하도록 하겠습니다.
    • 코드 동작 확인 : hub.push() 구문의 경우에는 테스트시 권한이 없어서 실패 되는 부분은 정상이라고 판단하였습니다.

@teddylee777
Copy link
Contributor

@musangk 님 양질의 리뷰 감사드립니다!

@teddylee777 teddylee777 merged commit 80978ce into LangChain-OpenTutorial:main Jan 4, 2025
1 check failed
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