Merge branch 'main' into akran123

Author: swthewhite

.github/workflows/check-function-length.yml

@@ -18,6 +18,9 @@ jobs:
           echo "max-locals=10" >> .pylintrc
           echo "max-args=5" >> .pylintrc
           echo "max-statements=15" >> .pylintrc
+          echo "[MESSAGES CONTROL]" >> .pylintrc
+          echo "disable=all" >> .pylintrc
+          echo "enable=design" >> .pylintrc
 
       - name: Run pylint
         run: pylint --rcfile=.pylintrc src/

.github/workflows/check-no-external-libs.yml

@@ -22,19 +22,25 @@ jobs:
 
           allowed_modules = {'sys', 'os', 'math', 'random', 'datetime', 're', 'enum'}
 
+          def is_internal_module(module_name):
+              \"\"\" 내부 모듈(`src/` 폴더 내 Python 파일)인지 확인 \"\"\"
+              module_path = os.path.join('src', module_name.replace('.', '/') + '.py')
+              module_dir = os.path.join('src', module_name.replace('.', '/'))
+              return os.path.exists(module_path) or os.path.isdir(module_dir)
+
           def check_imports(file_path):
               with open(file_path, 'r', encoding='utf-8') as f:
                   tree = ast.parse(f.read(), filename=file_path)
                   for node in ast.walk(tree):
                       if isinstance(node, ast.Import):
                           for alias in node.names:
-                              if alias.name not in allowed_modules:
+                              if alias.name not in allowed_modules and not is_internal_module(alias.name):
                                   raise SystemExit(f'❌ External library detected: {alias.name} in {file_path}')
                       elif isinstance(node, ast.ImportFrom):
-                          if node.module and node.module not in allowed_modules:
+                          if node.module and node.module not in allowed_modules and not is_internal_module(node.module):
                               raise SystemExit(f'❌ External library detected: {node.module} in {file_path}')
 
           for root, _, files in os.walk('src'):
               for file in files:
                   if file.endswith('.py'):
-                      check_imports(os.path.join(root, file))"
+                      check_imports(os.path.join(root, file))"

pytest.ini

@@ -1,4 +1,4 @@
 [pytest]
 pythonpath = src
 markers =
-    custom_name: 테스트 설명을 위한 커스텀 마커
+    custom_name: 테스트 설명을 위한 커스텀 마커

requirement.txt requirements.txtrequirement.txt renamed to requirements.txt

@@ -1,4 +1,19 @@
-from .lotto import Lotto
+# src/lotto/__init__.py
 
+# 📌 이 패키지는 로또 관련 기능을 제공하는 모듈입니다.
+# 외부에서 `from lotto import Lotto`와 같은 방식으로 사용할 수 있도록 
+# 필요한 모듈을 여기에 등록하세요.
+#
+# ✅ 새로운 모듈을 추가할 경우:
+# - `from .[모듈명] import [클래스/함수]` 형식으로 추가하세요.
+# - 필요한 경우 `__all__`에 추가하여 패키지 외부에서 명확하게 사용할 수 있도록 정의하세요.
+# - `flake8`의 F401 경고(`imported but unused`)가 발생하는 경우, `__all__`을 활용해 해결하세요.
 
-__all__ = ["Lotto"]
+from .lotto import Lotto  # 🎲 로또 번호 생성 및 검증을 위한 클래스
+
+# 패키지 외부에서 `from lotto import *` 사용 시 제공할 모듈을 명시적으로 정의합니다.
+__all__ = ["Lotto"]
+
+# 💡 예시: 새로운 모듈을 추가할 때
+# from .other_module import OtherClass  # 🆕 예: 새로운 클래스 추가 시
+# __all__.append("OtherClass")  # `__all__`에 추가하여 외부에서 접근 가능하게 함.