#11 sonar scanner 설정을 sonar-project.properties 에서 하도록 변경 #48
Conversation
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
4 similar comments
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
2 similar comments
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
| -Dsonar.pullrequest.github.repository=${{ github.repository }} \ | ||
| -Dsonar.sources=api/src \ | ||
| -Dsonar.java.binaries=api/build/classes \ | ||
| -Dsonar.log.level=INFO \ |
There was a problem hiding this comment.
이게 소나 큐브 프로젝트 프로퍼티로 옮겨지면 어떻게 되는건가요?
There was a problem hiding this comment.
sonar-scanner-action 이 sonar-project.properties 읽어 동일한 기능을 하게 됩니다.
기존 불필요한 설정값들이 있는 것 같아 필요한 것만 필터링하였는데요.
사용하면서 필요한 설정값을 추가하려고 합니다,
chohanjoo
changed the title
| -Dsonar.pullrequest.github.repository=${{ github.repository }} \ | ||
| -Dsonar.sources=api/src \ | ||
| -Dsonar.java.binaries=api/build/classes \ | ||
| -Dsonar.log.level=INFO \ |
.github/workflows/build.yml
Outdated
| GITHUB_TOKEN: ${{ secrets.TOKEN }} # Needed to get PR information, if any | ||
| run: ./gradlew jib --image=ghcr.io/${{ github.repository }}:${{ github.sha }} sonar --info | ||
| run: ./gradlew jib --image=ghcr.io/${{ github.repository }}:${{ github.sha }} --info |
There was a problem hiding this comment.
--image 옵션을 제거하니 아래 에러로 빌드가 실패합니다.
--info 옵션은 불필요해보여 제거하였습니다.
* What went wrong:
Execution failed for task ':api:jib'.
> com.google.cloud.tools.jib.plugins.common.BuildStepsExecutionException: Build image failed, perhaps you should make sure your credentials for 'registry-1.docker.io/library/api-unspecified' are set up correctly. See https://github.com/GoogleContainerTools/jib/blob/master/docs/faq.md#what-should-i-do-when-the-registry-responds-with-unauthorized for help
There was a problem hiding this comment.
core 에서 image 설정해줘서 jib 로 충분하다고 말씀하신걸까요?
There was a problem hiding this comment.
최신 core version 이랑 현 pr 기준 core version 이랑 변경사항이 커서 변경해야 할 사항이 많을 것 같습니다.
아래 pr 에서 core version 을 최신껄로 올렸는데요.
2개 pr 머지후에 다음 pr 로 반영해도 될까요? ㅎㅎ
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
Changes
Reason for change
Detailed Description
mention
@always0ne
Resolves: #
See also: #