단어 정보를 가져와 DB에 저장하는 함수 추가 #18

[kimdoori]

주요 변경 사항

• 우리말샘 API 셋팅 관련 코드 추가
• word의 target_code를 unique key로
• 단어 정보를 가져와 DB에 저장하는 API 추가 우리말샘 API를 사용하여 단어 정보 가져와서 저장하기 #18
  • 유의어는 wordnet을 이용해야할 것 같아서 이 PR에는 포함하지 않았습니다.

DB

pronunciation

word

sentence

word_sentenct_assoc

[kimdoori]

#24 (comment)
#24 (comment)
문서화

빼고 다 반영했습니다. 나머지는 일단 슬랙에서 말한.. 그 에러.. 해결하고 여기서 반영할게여 흑..

[kimdoori]

@zoripong 다 반영했습니다! (아마도..ㅎ) 그리고 앞으로 리뷰 해주시는 사항은 작업이 늘어지는 것을 막기 위해(ㅠㅠ) 크리티컬하거나 간단한 수정이 아니면 (즉, 반드시 이 PR에서 고쳐야하는 것이 아니면) 다음 이슈로 넘기려고 합니다. 찬성하시나요? (ex 문서화, )

[zoripong]

다 반영했습니다! (아마도..ㅎ) 그리고 앞으로 리뷰 해주시는 사항은 작업이 늘어지는 것을 막기 위해(ㅠㅠ) 크리티컬하거나 간단한 수정이 아니면 (즉, 반드시 이 PR에서 고쳐야하는 것이 아니면) 다음 이슈로 넘기려고 합니다. 찬성하시나요? (ex 문서화, )

작업이 늘어지는 것에 대해서는 저도 좋지 않다고 생각해요.
그런데 반드시 이 PR에서 고쳐야하는 것이 아니면 이 기준은 작업자에 따라서 다르다고 생각하기 때문에 리뷰어와의 커뮤니케이션은 필요할 것 같아요.
좀 더 핑퐁 시간을 줄이기 위해서 리뷰를 다는 사람이 다음 PR에서 해도 좋다. 라는 의견과 함께 남겨도 좋을 것 같네요.
다만 한 가지 걱정이 드는 것은 다음 PR에서 진행한다고 이야기 하고 진행되지 않는 건들이 생길 수 있을 것 같네요. 🤔

[kimdoori]

다 반영했습니다! (아마도..ㅎ) 그리고 앞으로 리뷰 해주시는 사항은 작업이 늘어지는 것을 막기 위해(ㅠㅠ) 크리티컬하거나 간단한 수정이 아니면 (즉, 반드시 이 PR에서 고쳐야하는 것이 아니면) 다음 이슈로 넘기려고 합니다. 찬성하시나요? (ex 문서화, )

작업이 늘어지는 것에 대해서는 저도 좋지 않다고 생각해요.
그런데 반드시 이 PR에서 고쳐야하는 것이 아니면 이 기준은 작업자에 따라서 다르다고 생각하기 때문에 리뷰어와의 커뮤니케이션은 필요할 것 같아요.
좀 더 핑퐁 시간을 줄이기 위해서 리뷰를 다는 사람이 다음 PR에서 해도 좋다. 라는 의견과 함께 남겨도 좋을 것 같네요.
다만 한 가지 걱정이 드는 것은 다음 PR에서 진행한다고 이야기 하고 진행되지 않는 건들이 생길 수 있을 것 같네요. 🤔

넴 동의하시면 당연히 그렇게 할 예정이었습니당! 이건 다음 PR에서 진행하는 건 어떨까요? 정도로.. 근데 그냥 리뷰어가 먼저 달아도 되겠네여. 걱정하신 부분은,, 저도 걱정이네여..

[zoripong]

다 반영했습니다! (아마도..ㅎ) 그리고 앞으로 리뷰 해주시는 사항은 작업이 늘어지는 것을 막기 위해(ㅠㅠ) 크리티컬하거나 간단한 수정이 아니면 (즉, 반드시 이 PR에서 고쳐야하는 것이 아니면) 다음 이슈로 넘기려고 합니다. 찬성하시나요? (ex 문서화, )

작업이 늘어지는 것에 대해서는 저도 좋지 않다고 생각해요.
그런데 반드시 이 PR에서 고쳐야하는 것이 아니면 이 기준은 작업자에 따라서 다르다고 생각하기 때문에 리뷰어와의 커뮤니케이션은 필요할 것 같아요.
좀 더 핑퐁 시간을 줄이기 위해서 리뷰를 다는 사람이 다음 PR에서 해도 좋다. 라는 의견과 함께 남겨도 좋을 것 같네요.
다만 한 가지 걱정이 드는 것은 다음 PR에서 진행한다고 이야기 하고 진행되지 않는 건들이 생길 수 있을 것 같네요. 🤔

넴 동의하시면 당연히 그렇게 할 예정이었습니당! 이건 다음 PR에서 진행하는 건 어떨까요? 정도로.. 근데 그냥 리뷰어가 먼저 달아도 되겠네여. 걱정하신 부분은,, 저도 걱정이네여..

다음 PR에서 진행한다면 이슈로 만들어두기,,? 그리고 리뷰가 진행됐던 사람이 아니더라도 누군가 가져가서 진행하기,,? (이러다 이슈가 1000만개가 되었다는,,그런 이야기,,)

[zoripong]

머지하기 전까지는 리뷰 반영 사항을 커밋으로 남겨두지 않고 해당 작업 내역의 커밋에 합치는게 더 좋지만,, 이것은 선택사항으로 해요,,

[zoripong]

리뷰가 길어지고 있지만 초반이라 그런 것도 있는 것 같아욤 거의 끝이 보이니 좀 만 더 ,, 👊🔥

[kimdoori]

머지하기 전까지는 리뷰 반영 사항을 커밋으로 남겨두지 않고 해당 작업 내역의 커밋에 합치는게 더 좋지만,, 이것은 선택사항으로 해요,,

저는 린트나 리뷰 반영으로 인해 결과가 달라지는 로직 변경 같은 것은 작업 커밋과 합치는 편이고, 네이밍 변경이나 구조 변경, 리뷰를 반영해도 결과는 변하지 않는 로직 변경 같은 변경 사항은 커밋 메시지를 붙여(리뷰 사항 반영 commit <- 이런 거 XXX) 새로운 커밋으로 두는 편입니다!

[kimdoori]

리뷰가 길어지고 있지만 초반이라 그런 것도 있는 것 같아욤 거의 끝이 보이니 좀 만 더 ,, 👊🔥

ㅠㅜ.... python 을 많이 만져보지도 않았고, 만져본지 좀 지나서 자꾸 뭘 빠트리네여ㅠ 담엔 더 신경써볼게여... 쥬륵..

[zoripong]

✨수🔥고🎁많👊으🤖셨🙏어🍏요👻

[kimdoori]

@zoripong 진짜 끝ㄲ그그ㅡ트ㅡㄱ끝!!!!!

[zoripong]

✨수🔥고🎁많👊으🤖셨🙏어🍏요👻