Skip to content

Refactor/#004 tombstone 처리 추가 1 #10

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 14 commits into from
Jan 15, 2025
Merged

Refactor/#004 tombstone 처리 추가 1 #10

merged 14 commits into from
Jan 15, 2025

Conversation

@hyonun321
Copy link
Collaborator

@hyonun321 hyonun321 commented Jan 15, 2025 •
edited
Loading

📝 변경 사항

  • Tombstone 처리 추가 #4

  • TombStone 개념 추가

    • deleted 플래그 Node에 추가
    • LinkedList에 deleteNode 메서드, insertById, findByIndex 등의 메서드 수정
    • JEST로 LinkedList각 메소드 테스트 코드 작성

  • CRDT라이브러리 폴더구조 변경에 따른 tsconfig, vite.config, webpack config 등등 수정

  • TombStone 가비지 컬렉트 메소드 작성 (서버쪽)

🔍 변경 사항 설명

  • 하드 딜리트 -> 소프트 딜리트로 변경됨에 따라 메소드들 동작하게 확인 및 테스트

🙏 질문 사항

  • 리뷰어에게 부탁하고싶은 체크리스트를 추가합니다.

📷 스크린샷 (선택)

  • UI 변경이 있는 경우 스크린샷이나 GIF를 첨부합니다.

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

hyonun321 and others added 8 commits January 14, 2025 18:10
@hyonun321 @Ludovico7 @minjungw00
build: 폴더 경로 변경에 따라 각 config 파일들 설정 변경
b3b6651 
- noctaCRDT -> noctaCRDT/src 등으로 구조 변경
- tsconfig
- vite
- webpack등의 경로 수정

Co-authored-by: Yeonkyu Min <Ludovico7@users.noreply.github.com>
Co-authored-by: minjungw00 <minjungw00@naver.com>
@hyonun321
refactor: noctaCrdt 라이브러리 구조변경
35c09cd 
- src파일에 주요 파일들 이동
- Interface의경우 types로 따로이동
@hyonun321
test: LinkedList 메서드 테스트 코드 작성
8038505 
- TombStoen 적용에 따른 메소드 체크
- 그외 기본동작 메소드 체크
@hyonun321
refactor: 경로 변경에 따른 import 수정
5da15a7
@hyonun321
feat: TombStone 가비지 컬렉팅 함수 추가
c897718 
- clearDeleteObject 메소드 서버쪽에 추가
@hyonun321
refactor: TombStone 역직렬화 과정에서 누락된 deleted 추가
4f7b642
@hyonun321
refactor: 가비지 컬렉팅 적용
b5126c9 
- DB에 node들 저장되지 않게 처리
@hyonun321
Merge branch 'dev' of https://github.com/boostcampwm-2024/refactor-we…
18ba1a0 
…b33-Nocta into Refactor/#004_Tombstone_처리_추가
@hyonun321 hyonun321 added Refactor refactor codes FE Front End BE Back End Test Test some metrics and measurement labels Jan 15, 2025
@hyonun321 hyonun321 self-assigned this Jan 15, 2025
Ludovico7
Ludovico7 approved these changes Jan 15, 2025
View reviewed changes
Copy link
Collaborator

@Ludovico7 Ludovico7 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@hyonun321
Copy link
Collaborator Author

hyonun321 commented Jan 15, 2025

Workspace 가아니라 WorkSpace S의 대문자 문제로 pnpm test가 진행이 안됐다. 추후에 Workspace로 통일해야함.

pipisebastian
pipisebastian approved these changes Jan 15, 2025
View reviewed changes
Copy link
Member

@pipisebastian pipisebastian left a comment

Choose a reason for hiding this comment

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

수고하셨숩니다용!!!

@github-actions github-actions bot merged commit 3141337 into dev Jan 15, 2025
5 checks passed
@minjungw00 minjungw00 linked an issue Jan 15, 2025 that may be closed by this pull request
Tombstone 처리 추가 #4
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pipisebastian pipisebastian pipisebastian approved these changes

@Ludovico7 Ludovico7 Ludovico7 approved these changes

@minjungw00 minjungw00 Awaiting requested review from minjungw00

Assignees

@minjungw00 minjungw00

@Ludovico7 Ludovico7

@hyonun321 hyonun321

Labels

BE Back End FE Front End Refactor refactor codes Test Test some metrics and measurement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Tombstone 처리 추가

5 participants

@hyonun321 @pipisebastian @Ludovico7 @minjungw00