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.

Sign up for GitHub

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

Jump to bottom

fix code style for namespace #2639

Merged

moreal merged 1 commit into development from feature/fix-code-style-namespace

Nov 20, 2024

+6 −2

Member

eugene-doobu commented Nov 20, 2024 •

edited

Loading

lib9c/headless등에 적용된 StyleCop 코드 규칙에 맞게 코드 변경을 진행 후 추가적으로 수정되는 스타일입니다.
StyleCop 규칙에서 using문에 namespace를 작성할 때 무조건 namespace의 풀 네임을 적용해야 하는 규칙이 있습니다
이를 대비하기 위해 라이더에서 'PreferQualifiedReference' 옵션을 True로 설정해두었는데 이로 인해 필드의 클래스를 지정할때에도 자동적으로 full namespace가 들어가는 케이스가 있고 이게 신경쓰이는 상황이 반복되어 False로 옵션을 변경해두었습니다

SA1210: Using directives should be ordered alphabetically by the namespaces.
SA1135: Using directive for namespace 'Lib9c.Tests.Action' should be qualified

현재 development를 머지 타겟으로 설정해두었는데, 현재 작업중인 250(클라기준) 패치와 관련된 작업 브랜치에 머지해야겠다 싶으시면 코멘트 남겨주세요!


          fix code style for namespace

3f99199

auto-assign bot requested review from area363, ipdae and moreal

November 20, 2024 01:36

auto-assign bot assigned eugene-doobu

moreal approved these changes

View reviewed changes

moreal merged commit 23ef6c6 into development

16 checks passed

eugene-doobu deleted the feature/fix-code-style-namespace branch

November 20, 2024 04:08

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet