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

Skip to content

Conversation

eunhhyy
Copy link
Contributor

@eunhhyy eunhhyy commented Jan 9, 2025

{PR Message - Write freely, remove this bracket if unnecessary}

Author Checklist

  • [v] PR Title Format: I have confirmed that the PR title follows the correct format. (e.g., [N-2] 07-Text Splitter / 07-RecursiveCharacterTextSplitter)

  • [v] Committed Files: I have ensured that no unnecessary files (e.g., .bin, .gitignore, poetry.lock, pyproject.toml) are included. These files are not allowed.

  • [v] (Optional) Related Issue: If this PR is linked to an issue, I have referenced the issue number in the PR message. (e.g., Fixes Update 01-PromptTemplate.ipynb #123)

  • ❌ Do not include unnecessary files (e.g., .bin, .gitignore, poetry.lock, pyproject.toml) or other people's code. If included, close the PR and create a new PR.

Review Template (Intial PR)

🖥️ OS: Win/Mac/Linux   
✅ Checklist      
 - [v] **Template**: Tutorials follows the required template. 
 - [v] **Table of Contents(TOC) Links**: All Table of Contents links work. ((Yes/No)
 - [v] **Image**: Image filenames follow guidelines.
 - [v] **Imports*: All import statements use the latest versions. Ensure "langchain-teddynote" is not used. 
 - [v] **Code Execution**: Code runs without errors.
 - Comments: {Write freely, 한국어 기술 가능}     

#212 (comment)

@eunhhyy eunhhyy mentioned this pull request Jan 9, 2025
8 tasks
Copy link
Contributor

@chaeyoonyunakim chaeyoonyunakim left a comment

Choose a reason for hiding this comment

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

2명 승인 조건을 위해 번역팀 PM님 추가합니다

@chaeyoonyunakim chaeyoonyunakim added the proofreading 번역/검수팀 제안사항 반영 label Jan 9, 2025
@teddylee777 teddylee777 merged commit 075ce5c into LangChain-OpenTutorial:main Jan 10, 2025
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proofreading 번역/검수팀 제안사항 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants