Skip to content

[Refactor] NeoImage 리펙토링 및 부분 Migration #115

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
Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

[Refactor] NeoImage 리펙토링 및 부분 Migration #115

wants to merge 10 commits into from

Conversation

@NeoSelf1
Copy link
Contributor

@NeoSelf1 NeoSelf1 commented Mar 9, 2025

✅ 작업 사항

  • NeoImage 내부 불필요 파일들을 삭제하고, Swift 6 동시성 모델을 일부 적용했습니다. 델리게이트와 콜백패턴을 사용하는 Delete, SessionDataTask, 그리고 성능 오버헤드의 중요성이 큰 MemoryStorage의 경우 현행유지하였으며, 나머지는 Sendable을 준수하거나 actor 타입으로 변경하였습니다.

😀 기타

  • 아직 ImageTask 연결은 미구현되어있습니다.

@NeoSelf1 NeoSelf1 self-assigned this Mar 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seongto seongto Awaiting requested review from seongto

@ericKwon95 ericKwon95 Awaiting requested review from ericKwon95

@tjdrb3807 tjdrb3807 Awaiting requested review from tjdrb3807

@generalban generalban Awaiting requested review from generalban

At least 2 approving reviews are required to merge this pull request.

Assignees

@NeoSelf1 NeoSelf1

Labels

refactor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@NeoSelf1