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.

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

[REFACTOR] 소나큐브를 각 pr 별로 확인할 수 있는 구조로 수정하기 #300

Closed
wants to merge 1 commit into from
Closed

[REFACTOR] 소나큐브를 각 pr 별로 확인할 수 있는 구조로 수정하기 #300

wants to merge 1 commit into from

Conversation

giantim
Copy link
Collaborator

@giantim giantim commented Nov 5, 2020 •
edited
Loading

PR 내용:

  • 현재 소나큐브는 배포 시 정적분석을 진행하도록 설정되어 있다.
  • pr을 보냈을 때도 정적 분석이 적용 되도록 변경했다.
  • 현재 테스트 pr 이기 때문에 pr을 여러번 보내면서 수정할 예정.
  • 테스트가 완료되면 라벨에 review needed를 달아놓겠습니다.

특이 사항 및 트러블 슈팅:

프로젝트 외부 설정:

closed: #284

@giantim giantim added ⚙️ Infra 인프라 관련 기능 🛠 Refactoring 기존 코드 개선 작업 labels Nov 5, 2020
@giantim giantim self-assigned this Nov 5, 2020
pci2676
pci2676 approved these changes Nov 6, 2020
View reviewed changes
Copy link
Collaborator

@pci2676 pci2676 left a comment

Choose a reason for hiding this comment

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

PR 빌드가 실패한것 같은데 그부분만 확인해주세요

@giantim giantim closed this Dec 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pci2676 pci2676 pci2676 approved these changes

@DunDung DunDung Awaiting requested review from DunDung

@moonyoungCHAE moonyoungCHAE Awaiting requested review from moonyoungCHAE

@ordinCode ordinCode Awaiting requested review from ordinCode

Assignees

@giantim giantim

Labels
⚙️ Infra 인프라 관련 기능 🛠 Refactoring 기존 코드 개선 작업
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[REFACTOR] 소나큐브를 각 pr 별로 확인할 수 있는 구조로 수정하기
2 participants
@giantim @pci2676