Skip to content

Feature/crawl suncare#15

Merged
jjunier merged 33 commits into
mainfrom
feature/crawl-suncare
Jul 15, 2025
Merged

Feature/crawl suncare#15
jjunier merged 33 commits into
mainfrom
feature/crawl-suncare

Conversation

@goosebumpsu

Copy link
Copy Markdown
Contributor

📌 PR 개요

해당 PR이 포함하는 주요 변경 사항 또는 목적을 간략히 설명해주세요.

  • 선케어 랭킹, 브랜드 데이터 크롤링 파일 업로드
  • 해당 파일로 로컬 Docker Airflow DAG 정상 실행 확인
  • 랭킹 데이터 내 카테고리 수집 코드 추가

✅ PR 체크리스트

코드 품질과 협업 효율을 위해 병합 전에 다음의 질문을 기반으로 점검해주세요.

  • 코드가 잘 실행돼요 (에러 없어요)
  • 내 변경사항이 다른 파트에 영향을 주지 않아요 (가능하면!)
  • 팀에서 정한 코드 스타일과 커밋 컨벤션을 잘 따랐어요
  • 관련 이슈나 설명이 필요한 내용은 아래에 적었어요

✏️ 주요 변경 사항

핵심 변경들을 자유롭게 적어주세요.

  • ERD에 작성한 컬럼 데이터 모두 수집하도록 작성
  • ruff 적용
  • docs/ruff-guide.md 변경사항 적용
  • .gitignore 적용
  • Dockerfile, requirements.txt 등 충돌 해결

🧪 📷 스크린샷 or 테스트 결과 (선택)

이해에 도움될 스크린샷을 첨부해주세요.

📎 관련 이슈

관련된 이슈 번호를 적어주세요. (예: #12, JIRA-23 등)

💬 리뷰어에게 하고 싶은 말 (선택)

설명이 필요한 부분, 고민했던 점, 리뷰 요청 포인트 등 자유롭게 적어주세요.

goosebumpsu and others added 30 commits July 3, 2025 14:04
@goosebumpsu goosebumpsu requested a review from jjunier as a code owner July 15, 2025 05:42

@jjunier jjunier left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

좋습니다!! 약간 아쉬운 점은 ruff 린터가 적용 명령어 ruff format {파일명}를 실행한 뒤 push가 된 건지 의문이 드네요. 그래도 코드상 문제점은 보이지 않아 승인하겠습니다.

@jjunier jjunier merged commit db992f1 into main Jul 15, 2025
2 checks passed
@jjunier jjunier assigned jjunier and goosebumpsu and unassigned jjunier Jul 15, 2025
@jjunier jjunier added good first issue Good for newcomers enhancement New feature or request labels Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants