Skip to content

Feature/crawl skincare#9

Merged
jjunier merged 29 commits into
mainfrom
feature/crawl-skincare
Jul 10, 2025
Merged

Feature/crawl skincare#9
jjunier merged 29 commits into
mainfrom
feature/crawl-skincare

Conversation

@sulsikan

@sulsikan sulsikan commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

📌 PR 개요

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

  • 스킨케어 랭킹, 브랜드 데이터 수집 파일 추가
  • 로컬 docker 환경 airflow DAG 테스트 통과

✅ PR 체크리스트

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

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

✏️ 주요 변경 사항

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

  • ERD에 작성한 컬럼의 데이터를 모두 수집하도록 수정
  • docker 환경에서도 실행 가능하도록 selenium 드라이버 부분을 수정
  • from airflow.utils.log.logging_mixin import LoggingMixin을 통해 Airflow의 내장 로깅 시스템을 사용하였으며, 이를 통해 로그가 Airflow 웹 UI 및 로그 파일에 자동 출력되도록 설정
  • ruff 적용
  • docs/ruff-guide.md 변경사항 적용

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

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

📎 관련 이슈

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

close #7
close #8

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

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

sulsikan and others added 24 commits July 1, 2025 11:11
@sulsikan sulsikan requested a review from jjunier as a code owner July 9, 2025 12:04
@sulsikan sulsikan closed this Jul 9, 2025
@jjunier

jjunier commented Jul 9, 2025

Copy link
Copy Markdown
Member

PR Check 단계에서 Ruff 기반 Code Lint 형식애 맞지 않는 코드가 발견되었나 보네요.

그리고 docs/Ruff-guide.md 문서도 이전에 수정 사항이 있었는데 크롤링 feature branch에 반영이 안 된 것도 확인이 됩니다.
→ 이 부분은 제가 적용해서 업데이트 해놓겠습니다.

@jjunier jjunier added good first issue Good for newcomers enhancement New feature or request labels Jul 9, 2025
@sulsikan

Copy link
Copy Markdown
Contributor Author

ruff로 코드 형식 체크진행과 docs/Ruff-guide.md 문서 수정사항 변경 적용했습니다!

@sulsikan sulsikan reopened this Jul 10, 2025
@sulsikan sulsikan closed this Jul 10, 2025
@sulsikan sulsikan reopened this Jul 10, 2025

@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.

.gitignore 파일 세팅이 안 되어 있었군요.. 제가 이런 실수를 하다니..

얼른 작업 후에 main으로 병합 해둘테니 잠시 PR 보류 바랄게요..

죄송합니다 🙏🙏

@sulsikan

Copy link
Copy Markdown
Contributor Author

.gitignore 파일도 적용됐습니다~!

@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.

크롤링 파트 수고하셨습니다~!

@jjunier jjunier merged commit 6a9e905 into main Jul 10, 2025
1 check passed
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.

Docker 크롬 브라우저 관련 에러 docker-compose 웹 크롤링 관련 이슈

2 participants