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

[fix] 프로젝트 세팅 일부 수정 #23

Merged
merged 3 commits into from
May 19, 2024
Merged

[fix] 프로젝트 세팅 일부 수정 #23

merged 3 commits into from
May 19, 2024

Conversation

tkdwns414
Copy link
Contributor

Related issue 🛠

Work Description ✏️

  • �gitignore에 application.yml 추가했습니다.
  • application.yml을 깃에서 삭제했습니다
  • Build가 잘 되는지, 테스트 결과를 확인하기 위한 CI workflow를 추가했습니다.

Uncompleted Tasks 😅

  • yaml을 secret이 아닌 submodule로 관리하기

To Reviewers 📢

CI workflow를 짜기는 했지만 저게 정석적인 형태인지까지는 확신하지 못하겠습니다.

빌드 시에 테스트 통과 검사까지 하려면 application.yml에 올바른 데이터베이스 정보가 저장돼있어야 하는데 이를 위해 github actions에서 제공하는 서비스를 통해 DB를 연결했습니다.

RDS의 경우 AWS에서 RDS에 public ipv4를 달면 요금이 부과되게 변경함을 통해 연결된 EC2 내부에서만 RDS에 접속할 수 있어 사용하지 못했습니다.

Copy link
Contributor

@kmjenny kmjenny left a comment

Choose a reason for hiding this comment

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

확인했습니당 ദ്ദി ᐕ))?

Copy link
Contributor

@minwoo0419 minwoo0419 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@tkdwns414 tkdwns414 merged commit 3714941 into develop May 19, 2024
1 check passed
@tkdwns414 tkdwns414 deleted the fix/22 branch May 19, 2024 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] 프로젝트 세팅 일부 수정
3 participants