Skip to content
This repository was archived by the owner on Jul 21, 2024. It is now read-only.

내부 변수 표기 방법, 디렉토리 구조 고도화, gitIgnore 변경 등 프로젝트 Style 수정 #53

Merged
merged 4 commits into from
Aug 22, 2023

Conversation

98StarJune
Copy link
Member

@98StarJune 98StarJune commented Aug 22, 2023

변수 표기 방법 변경

기존 코드에서는 DI 또는 클래스명과 동일한 변수를 선언할 때 m을 붙였으나, 예제/공식 문서 등에서는 underbar(_)가 많이 사용됨에 따라 이 부분도 동일하게 적용하도록 변경합니다.

변경 예시

기존 : mContext
변경 : _context

DTOs 내부 디렉토리 구조 변경

많은 DTO가 생성됨에 따라 DTO 구분을 쉽게하기 위해서 기존 Auth에 사용되던 DTO들은 DTOs/Auth 디렉토리로 이동되었습니다. 이에 따른 리팩토링이 실행되었습니다. 추후 개발 중 Chat관련 DTO는 DTOs/Chat 내부 디렉토리로 이동 바랍니다.

gitIgnore 파일 변동

마이그레이션 시 발생하는 디렉토리 (server/Data/) 디렉토리에 대한 gitIgnore이 추가되었습니다.

@98StarJune 98StarJune requested a review from a team August 22, 2023 06:18
@98StarJune 98StarJune changed the base branch from master to develop August 22, 2023 06:19
@98StarJune 98StarJune added the wontfix 기능 혹은 문서의 수정을 요청하는 경우 label Aug 22, 2023
98StarJune and others added 3 commits August 22, 2023 16:01
Chat 관련 DTO는 DTOs/Chat에 생성해주세요.
Database 마이그레이션에 이용되는 /Data/ 디렉토리를 제외 시켰습니다.
@98StarJune 98StarJune changed the title 내부 변수에 대한 사용 시 표기 방법 변경 변수 내 표기 방법, 디렉토리 구조 고도화, gitIgnore 변경 Aug 22, 2023
@98StarJune 98StarJune changed the title 변수 내 표기 방법, 디렉토리 구조 고도화, gitIgnore 변경 변수 내 표기 방법, 디렉토리 구조 고도화, gitIgnore 변경 등 프로젝트 Style 수정 Aug 22, 2023
@98StarJune 98StarJune changed the title 변수 내 표기 방법, 디렉토리 구조 고도화, gitIgnore 변경 등 프로젝트 Style 수정 내부 변수 표기 방법, 디렉토리 구조 고도화, gitIgnore 변경 등 프로젝트 Style 수정 Aug 22, 2023
Copy link
Member

@Hun3431 Hun3431 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

@yejin-e yejin-e left a comment

Choose a reason for hiding this comment

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

확인했습니다.
수고하셨습니다.

@98StarJune 98StarJune merged commit f74807e into develop Aug 22, 2023
@98StarJune 98StarJune deleted the feature/style-change branch August 22, 2023 15:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
wontfix 기능 혹은 문서의 수정을 요청하는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants