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.

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

feat: 학교 버스 시간표 마이그레이션(Go → Java) #1290

Closed
wants to merge 8 commits into from

Conversation

ImTotem
Copy link
Contributor

@ImTotem ImTotem commented Feb 22, 2025

🔥 연관 이슈

🚀 작업 내용

  1. Go 코드 그대로 Java로 옮겼습니다.
  2. 버스 리팩터링 코드와 conflict를 방지하고, 버스 코드와 yaml 파싱을 위한 코드는는 별개라고 생각되어 중복되어도 따로 구현했습니다.
  3. 학기와 학기 기간?도 실행과 동시에 수정할 수 있도록 했습니다.
  4. yaml 파일은 jar과 함께 빌드하도록 구현했습니다. (KOIN_BATCH_V2처럼 yaml 수정 후 재배포 해줘야 합니다.) → yaml 파일의 nodes를 기록으로 남기기 위해서 일부러 이렇게 만들었습니다.
  5. 테스트는 시간 관계상 패스했습니다. (아무나 해주면 좋겠다...)

💬 리뷰 중점사항

@ImTotem ImTotem added the Team Campus 캠퍼스 팀에서 작업할 이슈입니다 label Feb 22, 2025
@ImTotem ImTotem self-assigned this Feb 22, 2025
@github-actions github-actions bot added the 기능 새로운 기능을 개발합니다. label Feb 22, 2025
Copy link

github-actions bot commented Feb 22, 2025

Unit Test Results

339 tests   338 ✔️  1m 44s ⏱️
  45 suites      1 💤
  45 files        0

Results for commit e3b23cd.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@dradnats1012 dradnats1012 left a comment

Choose a reason for hiding this comment

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

👍

public class SchoolBus {

@Id
@Field("_id")
Copy link
Contributor

Choose a reason for hiding this comment

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

_는 일부러 넣은건가요??

String content = Files.readString(Paths.get(path));
return yaml.loadAs(content, clazz);
}

Copy link
Contributor

Choose a reason for hiding this comment

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

개행 발견이요

Copy link
Contributor

Choose a reason for hiding this comment

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

다른 노선들은 다른 곳에 있는건가요??

Copy link
Contributor

@kih1015 kih1015 left a comment

Choose a reason for hiding this comment

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

최고

@ImTotem ImTotem closed this Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team Campus 캠퍼스 팀에서 작업할 이슈입니다 기능 새로운 기능을 개발합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BATCH] 학교 버스 시간표
3 participants