Skip to content

코드 리펙토링 #56

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

Merged
merged 3 commits into from
Nov 7, 2024
Merged

코드 리펙토링 #56

merged 3 commits into from
Nov 7, 2024

Conversation

Soju06
Copy link
Owner

@Soju06 Soju06 commented Nov 7, 2024

🛠️ PR Summary

🌟 요약

어떤 것이 변경되었나요? 간략히 설명해주세요.

EllipsisType을 활용하도록 변경 및 기존 Impl 명명을 Mixin으로 변경했습니다.

📊 주요 변경 사항

주요 변경 사항을 적어주세요.

  • 기존 프로토콜 및 ABC 클래스의 미구현 함수를 raise NotImplementedError에서 ...으로 변경하였습니다.

🎯 목적 및 영향

  • 목적: 왜 이 PR이 필요한가요?
    �파이썬의 기본 기능을 활용하도록 변경하였습니다.

  • 영향: 이 변경 사항이 어떤 영향을 미치나요?
    라이브러리 임포트를 줄일 수 있습니다.

@Soju06 Soju06 added the 기능 새로운 기능 또는 요청 label Nov 7, 2024
@Soju06 Soju06 self-assigned this Nov 7, 2024
@Soju06 Soju06 merged commit 6cb7321 into main Nov 7, 2024
1 check passed
@Soju06 Soju06 deleted the feature/code-refactoring branch November 7, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 새로운 기능 또는 요청
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant