Skip to content

score 기능 추가#131

Merged
byundojin merged 6 commits intomainfrom
new
Oct 21, 2025
Merged

score 기능 추가#131
byundojin merged 6 commits intomainfrom
new

Conversation

@byundojin
Copy link
Member

기존 develop branch에 breakpoint가 너무 많아서 main에서 빠르게 score기능만 구현 후 main에 merge합니다.
추가된 기능은 score입니다. score-board는 추가되지 않았고, 이전 논의와 다르게 Cursor의 바운더리에 score 포함시키는 방향으로 개발하였습니다.
지금 당장 점수를 추가 할 수 있는 interaction이 move 밖에 없어서 move만 추가했습니다

test는 작성하지 않았습니다. 추후 추가해야 합니다.
env변경되었습니다.

10월 22일에 merge 할 것 입니다. 그 전까지 comment 남기면 반영하겠습니다.

@byundojin byundojin requested a review from onee-only October 20, 2025 18:13
Copy link
Contributor

@onee-only onee-only left a comment

Choose a reason for hiding this comment

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

LGTM. 어차피 제가 이해하기론 이 PR이 프론트를 위한 빠른 배포인데, 테스트를 추후에 추가할거라면 차라리 여기서 하거나 아니면 안 하는게 나을 것 같습니다.

CI 실패하는거 보니 타입 에러라 빠르게 그거 고치고 다시 돌려서 안 망가지는지만 보면 되겠네요.

@byundojin
Copy link
Member Author

기존 Cursor를 직접 생성하던 부분이 있어서 socre에 default값 추가 했습니다.
receiver_add_score가 복사한 함수 이름 그대로여서 실패하던 case 고쳤습니다.
move에서 event publish를 한번만 하는지 확인하는 case 고쳤습니다.

@byundojin
Copy link
Member Author

지금 예전 코드 보니 breakpoint가 많아도 그냥 develop고쳐서 쓰는게 나을 것 같습니다. test추가는 진행하지 않고 혹시 모르니 todo 주석은 달아 놨습니다. comment 달아주신 의견 반영되었으니 merge 하겠습니다.

@byundojin byundojin merged commit f8f06bb into main Oct 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants