[GAMSS-39] chore: CI 파이프라인 및 detekt 정적 분석 도구 추가#2
Merged
Conversation
- GitHub Actions CI 워크플로우 추가 (develop PR / develop·main push / 수동 실행 트리거) - detekt, Android lint, unit test, release 빌드를 매트릭스로 병렬 검사 - detekt + detekt-formatting(ktlint 규칙) 전 모듈 적용 - detekt 설정 추가: Compose @composable 네이밍 예외, formatting 규칙 활성화 - detekt SARIF를 병합해 GitHub 코드 스캐닝에 업로드 (업로드 실패는 게이트 비차단)
- 설명 주석 제거 (build.gradle.kts, ci.yml, detekt.yml, AppResult.kt) - detekt 플러그인 id를 버전 카탈로그에서 단일 소싱 - 미사용 DetektCreateBaselineTask 설정 제거 (jvmTarget 중복 해소) - detekt.yml에서 중복된 style.MaxLineLength 규칙 제거
soyeonLee126
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
작업 개요
프로젝트에 CI 파이프라인을 도입하고 정적 분석 도구(detekt)를 붙였습니다. PR 시점에 코드 스타일/포맷, Android lint, 유닛 테스트, 릴리즈 빌드가 자동으로 검증되도록 하여 develop 브랜치 품질을 지키는 것이 목적입니다.
작업 유형
변경 사항
.github/workflows/ci.yml)develop대상 PR /develop·mainpush / 수동 실행(workflow_dispatch)concurrency로 동일 ref 중복 실행 취소, 최소 권한(contents: read, SARIF용security-events: write)build.gradle.kts,gradle/libs.versions.toml)config/detekt/detekt.yml): Compose@Composable네이밍 예외, formatting 규칙 활성화continue-on-error)core의 Result 래퍼(AppResult.of)에 의도된 광범위 catch를 위한@Suppress("TooGenericExceptionCaught")추가관련 이슈
관련 작업 (Notion)
스크린샷 / 동작 화면
UI 변경 없음 (CI/빌드 설정 작업)
체크리스트
develop으로 설정되어 있다feat:,fix:등)을 따른다리뷰 요청 사항
config/detekt/detekt.yml)와 CI 검사 범위가 팀 기준에 맞는지 봐주세요. 규칙은 이후 조정 가능합니다.