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

[ Ci/deploy ] Develop 브랜치 Server Mock API 사용 #49

Merged
merged 7 commits into from
Jun 16, 2024

Conversation

soomin9106
Copy link
Collaborator

@soomin9106 soomin9106 commented Jun 15, 2024

🔥 Related Issues

resolve #issue_number
close #issue_number

💜 작업 내용

  • Cloudflare 환경 변수 설정
  • gitignore 추가

✅ PR Point

  • 배포 환경에서는 runtime = edge 추가해야 합니다.
  • dotenv-cli 패키지 도입을 통해 local, development, production 나눠서 실행 가능하도록 구현했습니다.

Copy link
Collaborator

@Happhee Happhee left a comment

Choose a reason for hiding this comment

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

완전 굿굿!! 배포 어려운데 해줘서 고마오 숨니!

@soomin9106
Copy link
Collaborator Author

soomin9106 commented Jun 15, 2024

dotenv-cli 패키지 사용해서
pnpm dev:local
pnpm dev:development
pnpm dev:production
으로 환경 나누었습니당!! @Happhee

이 패키지는 일단 많이 사용하고 있는 패키지 같아서 도입해봤는데, 다른 패키지로 바꾸어도 될 것 같습니다!!
아니면 패키지로 관리할 필요가 없을까용? ㅋㅋ

@Happhee
Copy link
Collaborator

Happhee commented Jun 15, 2024

@soomin9106 오오!1 아니아니!! 패키지 도입해야지 ㅎㅎㅎ 나는 https://www.npmjs.com/package/env-cmd 요거 알고있긴했는데 용량 비교해보니까 숨니가 해준게 더 작다! 좋아욯ㅎㅎㅎ 혹시 environment 관리하는걸 최상위 폴더에 해주는건 어떄?!!

image

@soomin9106
Copy link
Collaborator Author

스크린샷 2024-06-15 오후 10 52 08 이렇게 넣으시면 됩니다요! (이 말이 맞지용?)

@Happhee
Copy link
Collaborator

Happhee commented Jun 15, 2024

오옹오!!마자요!!! 명령어도 변경해죠야할듛?!!

@soomin9106
Copy link
Collaborator Author

76105fb
이렇게 명령어 변경 했어!! 머지할게윱

@Happhee
Copy link
Collaborator

Happhee commented Jun 16, 2024

머지 해주세요! @soomin9106

@Happhee
Copy link
Collaborator

Happhee commented Jun 16, 2024

gitignore 파일에

dependencies

environments

이것도 추가해줘야 할것같다..!

@soomin9106 soomin9106 merged commit 31420bb into develop Jun 16, 2024
2 checks passed
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.

2 participants