Skip to content

31, 32항 테스트 케이스 추가#34

Closed
HanNayeoniee wants to merge 9 commits into
dev-five-git:mainfrom
HanNayeoniee:main
Closed

31, 32항 테스트 케이스 추가#34
HanNayeoniee wants to merge 9 commits into
dev-five-git:mainfrom
HanNayeoniee:main

Conversation

@HanNayeoniee
Copy link
Copy Markdown

@HanNayeoniee HanNayeoniee commented Aug 15, 2025 •
edited
Loading

31, 32항 테스트 케이스 추가

@HanNayeoniee
add rule_31.csv
3259d2f 
Add test rule 31 test case
@HanNayeoniee
Add rule_32.csv
6027d3e 
Add rule 32 test case
@HanNayeoniee
rule_map.json add rule 31, 32
7edbf9c
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Aug 15, 2025 •
edited
Loading

⚠️ No Changeset found

Latest commit: fe64860

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

owjs3901
owjs3901 requested changes Aug 15, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@owjs3901 owjs3901 left a comment

Choose a reason for hiding this comment

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

wrap 이슈일 경우 csv 파일 파싱에 실패한 것입니다
확인바랍니다

owjs3901
owjs3901 requested changes Aug 15, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@owjs3901 owjs3901 left a comment

Choose a reason for hiding this comment

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

wrap 이슈일 경우 csv 파일 파싱에 실패한 것입니다
확인바랍니다

owjs3901
owjs3901 requested changes Aug 15, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@owjs3901 owjs3901 left a comment

Choose a reason for hiding this comment

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

아직 오류가 있습니다 ㅠㅠ
main에서 pull 혹은 merge로 병합을 한 후 오류 메시지로 어떤 것에서 오류가 발생하는지 확인해보시는 것도 좋아보입니다

@HanNayeoniee
Copy link
Copy Markdown
Author

HanNayeoniee commented Aug 15, 2025

아직 오류가 있습니다 ㅠㅠ main에서 pull 혹은 merge로 병합을 한 후 오류 메시지로 어떤 것에서 오류가 발생하는지 확인해보시는 것도 좋아보입니다

merge 할 때 발생한 충돌이 아닌, 제가 추가한 테스트 케이스에 존재하는 오류인거죠?

@owjs3901
Copy link
Copy Markdown
Contributor

owjs3901 commented Aug 15, 2025

네 맞습니다

만약 해당 부분이 문제가 없다면 최소 다른 테스트 케이스에 대하여 총 몇 개가 맞았는지 수가 떠야 합니다!

@owjs3901 owjs3901 closed this Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@owjs3901 owjs3901 owjs3901 requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@HanNayeoniee @owjs3901