Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feat] 파일 읽고, GPT 호출하고, 저장하기 #9

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

codesejin
Copy link
Collaborator

@codesejin codesejin commented Aug 9, 2023

📌 PR Checklist

Please check if your PR fulfills the following requirements:

✨ PR Type

What kind of change does this PR introduce?

  • 🐛 Bugfix
  • 🚀 Feature
  • 📝 Code style update (formatting, local variables)
  • ♻️ Refactoring (no functional changes, no API changes)
  • 🛠️ Build related changes
  • 🔄 CI related changes
  • 📚 Documentation content changes
  • 🌐 angular.io application / infrastructure changes
  • 🆕 Other... Please describe:

🧑‍💻 What is the current behavior?

Issue Number:

관련된 이슈 넘버인데, 확인하고 싶으시다면 클릭해주세요.
#6 #8

커밋을 확인하고 싶다면 아래 링크를 클릭해주세요.
변경된 커밋 내용 : 커밋 해시 넘버 6257310

민규님이 알려주신 cli 관련 설정 참고할 만한 사이트 입니다
원본 : https://blog.logrocket.com/building-typescript-cli-node-js-commander/
번역본 : https://itchallenger.tistory.com/851

🚶‍♀️ What is the new behavior?

package.json , tsconfig.json , dummy.ts , api-caller.ts 등 변경 사항이 많습니다

기본 시작 명령어 node dist/dummy.js
파일 읽고, gpt 호출하고, 파일 저장하는 명령어 node dist/dummy.js g -f samples/user-data-config.json -o json

아직 더 수정해야할 것

  1. 코드상 문제는 없는거 같은데 로그가 2번씩 찍히고, 데이터 저장이 2번씩 되는 이유를 모르겠습니다.
  2. 사용자가 user-data-config.json을 텍스트편집기를 통해 수정할때, 저희가 정해놓은 템플릿 이외에 내용을 추가한다면 익셉션 처리 해야합니다
    (이유 : 의도했던 바랑 다르게 input값이 달라져서 gpt의 응답이 달라질 수 있음)

🚨 Does this PR introduce a breaking change?

  • ✅ Yes
  • ✅ No

📸 Other information

image

@pyuseon
Copy link
Collaborator

pyuseon commented Aug 10, 2023

고생많으셨습니다! 코드 분리 깔끔하게 잘 해주셨네요!
env 파일 문제는 제가 해결해서 pr 날리도록 하겠습니다 !

@pyuseon
Copy link
Collaborator

pyuseon commented Aug 10, 2023

아! 그리고 develop 브랜치에도 머지 부탁드립니다 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants