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

refactor: 환경변수 가져오는 함수 수정 #278

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Conversation

leeminhee119
Copy link
Member

💻 스크린샷 (선택사항)

📃 PR 설명

  • 환경변수를 불러오는 함수를 수정했습니다.
  • checkEnv -> getEnvVariable
    • 수정 전: import.meta.env 에서 먼저 구조분해로 환경변수를 읽은 후에 checkEnv를 통해 값이 undefined인지 확인
    • 수정 후: getEnvVariable 함수에 환경변수 이름을 전달하여 import.meta.env[환경변수 이름]이 undefined인지 확인
  • 내용은 사실상 전후 동일하지만, 사용하는 입장에서 좀 더 자연스럽게 읽히도록 수정했습니다.
  • 기존에는 새로운 환경변수 추가하려면 구조분해할당문과 checkEnv에 전달하는 argument 두 개를 작성해야 했다면, 변경된 버전에서는 argument 하나만 작성하면 됩니다.

📃 참고사항

🔎 PR포인트 및 궁금한 점

@leeminhee119 leeminhee119 added the 🛠 Refactor 리팩토링한 경우(리팩토링 과정에서 파일 삭제, 이동 포함) label Mar 6, 2024
@leeminhee119 leeminhee119 self-assigned this Mar 6, 2024
Copy link
Contributor

@backward99 backward99 left a comment

Choose a reason for hiding this comment

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

이해 했습니다! 수고하셨습니다!

@leeminhee119 leeminhee119 merged commit 189938e into main Mar 13, 2024
1 check passed
@leeminhee119 leeminhee119 deleted the fix/check-env branch March 13, 2024 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Refactor 리팩토링한 경우(리팩토링 과정에서 파일 삭제, 이동 포함)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants