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

Skip to content

Conversation

ErikaPark
Copy link
Contributor

@ErikaPark ErikaPark commented Jan 15, 2025

코드 결과물이 너무 길다는 우려가 있습니다.
튜토리얼의 특성상 요약이 점진적으로 개선되는 과정을 보여주는 것이 중요하다고 판단되어 현재는 그대로 두었는데요,
출력 방식을 더 효과적으로 조정할 수 있는 좋은 아이디어가 있다면 제안 부탁드립니다.

Author Checklist

  • PR Title Format: I have confirmed that the PR title follows the correct format.
  • Committed Files: I have ensured that no unnecessary files.

Copy link
Contributor

@jeong-wooseok jeong-wooseok left a comment

Choose a reason for hiding this comment

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

🖥️ OS: Win
✅ Checklist

  • Template: Tutorials follows the required template.
  • Table of Contents(TOC) Links: All Table of Contents links work. ((Yes/No)
  • Image: Image filenames follow guidelines.
  • *Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
  • Code Execution: Code runs without errors.
  • 코드 및 내용이 알차게 잘 작성된 것 같아서 유지하였습니다.
  • 통일성에 맞게 일부 양식 수정하였습니다.

@ErikaPark
Copy link
Contributor Author

  • 통일성에 맞게 일부 양식 수정하였습니다.

@jeong-wooseok 안녕하세요! 😊
새로 추가된 커밋이 보이지 않는데, 혹시 수정된 내용을 커밋해주실 수 있을까요?
양식 수정이 반영되지 않은 것 같아 확인 차 여쭤봅니다.

Copy link
Contributor

@sirena1 sirena1 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:

    • Windows
  • Template Rule 준수 여부:

    • Template Rule 가이드를 준수하였는가? (Check 표기)
  • Table of Contents 링크 확인:

    • Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)
  • 이미지 파일명 검토:

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

    • import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)
  • 코드 동작 확인:

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

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

@teddylee777 teddylee777 merged commit fd490a0 into LangChain-OpenTutorial:main Jan 18, 2025
1 of 2 checks passed
@sohyunwriter sohyunwriter added the docs tutorial label Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs tutorial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants