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: 소나클라우드 커버리지 대상 exclusions 범위 수정 #148

Merged
merged 3 commits into from
Dec 29, 2023

Conversation

Ji-soo708
Copy link
Member

@Ji-soo708 Ji-soo708 commented Dec 26, 2023

개요

  • 소나클라우드에서 커버리지 측정할 때, exception과 Response에 해당하는 부분들까지 포함하여 측정해서 해당 부분을 제대로 제외하도록 수정했습니다.

작업사항

  • 소나클라우드 커버리지 대상 exclusions 수정

주의사항

  • 이제 exception과 response는 측정에서 제외되는 거로 보이는데 config도 갑자기 측정돼서 이것도 다시 제외되도록 수정은 했습니다.

@Ji-soo708 Ji-soo708 self-assigned this Dec 26, 2023
@Ji-soo708 Ji-soo708 added the 인프라 인프라 및 데브옵스 세팅 label Dec 26, 2023
@Ji-soo708 Ji-soo708 changed the title fix: 소나클라우드 커버리지 대상 exclusions 수정 fix: 소나클라우드 커버리지 대상 exclusions 범위 수정 Dec 26, 2023
Copy link

sonarqubecloud bot commented Dec 26, 2023

Quality Gate Failed Quality Gate failed

Failed conditions

0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@Ji-soo708 Ji-soo708 requested review from jung-woo-kim and kth990303 and removed request for jung-woo-kim December 26, 2023 12:14
Copy link
Member

@kth990303 kth990303 left a comment

Choose a reason for hiding this comment

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

임시로 exclude 하는 것으로 들어서 approve 했습니다~

근데 new code에 code smell 이 0개여야만 code quality Failed가 안뜨더라고요. 좀 빡센 것 같은데 해당 문서 참고해서 기준을 변경하는 것도 좋아보여요

(추가로, 제가 어드민이 아니어서 그런지 권한이 없는데 혹시 권한 주실 수 있을까요? ㅜㅜ)
image

Comment on lines +73 to +74
property 'sonar.exclusions', '**/test/**, **/resources/**, **/*Application*.java, **/*Controller*.java, **/*Config.java' +
'**/*Response.java, **/*Exception.java, **/security/**, **/support/**, **/Q*.java'
Copy link
Member

Choose a reason for hiding this comment

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

추후에는 제외 안시키는 것도 괜찮아보여요~!
아래 사진은 config 파일에서 code smell 잡아낸 케이스인데, config, response 등 해당 파일들도 우리가 짜는 코드단이다보니 code smell 측정해주는 게 좋아보여서요
image

Copy link
Member Author

Choose a reason for hiding this comment

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

넵 추후 리팩터링하게 된다면 그때 제외하겠습니다~

@Ji-soo708
Copy link
Member Author

임시로 exclude 하는 것으로 들어서 approve 했습니다~

근데 new code에 code smell 이 0개여야만 code quality Failed가 안뜨더라고요. 좀 빡센 것 같은데 해당 문서 참고해서 기준을 변경하는 것도 좋아보여요

(추가로, 제가 어드민이 아니어서 그런지 권한이 없는데 혹시 권한 주실 수 있을까요? ㅜㅜ)

권한 드렸습니다! 저희 프로젝트에서도 sonar way 말고 mocacong 규칙 만들어서 새로 기준 잡아놨었는데 mocacong에서도 코드 스멜은 측정 제외되도록 했습니다. 케이님이 보시고 더 제외하고 싶은 기준이 있다면 제외하셔도 되세요!

@Ji-soo708 Ji-soo708 merged commit d6708ab into develop Dec 29, 2023
2 of 3 checks passed
@Ji-soo708 Ji-soo708 deleted the MOCACONG-494-sonar-exclusions branch December 29, 2023 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
인프라 인프라 및 데브옵스 세팅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants