Remove unnecessary nested try-except block for cleaner code. Streamlined the import statement to remove an unused reference to get_git_provider.

This commit is contained in:
Sangmin Park
2025-05-15 18:41:39 +09:00
parent b6a401bcc2
commit e55fd64bda

View File

@ -6,8 +6,7 @@ from dynaconf import Dynaconf
from starlette_context import context from starlette_context import context
from pr_agent.config_loader import get_settings from pr_agent.config_loader import get_settings
from pr_agent.git_providers import (get_git_provider, from pr_agent.git_providers import get_git_provider_with_context
get_git_provider_with_context)
from pr_agent.log import get_logger from pr_agent.log import get_logger
@ -16,12 +15,8 @@ def apply_repo_settings(pr_url):
git_provider = get_git_provider_with_context(pr_url) git_provider = get_git_provider_with_context(pr_url)
if get_settings().config.use_repo_settings_file: if get_settings().config.use_repo_settings_file:
repo_settings_file = None repo_settings_file = None
try:
try: try:
repo_settings = context.get("repo_settings", None) repo_settings = context.get("repo_settings", None)
except Exception:
repo_settings = None
pass
if repo_settings is None: # None is different from "", which is a valid value if repo_settings is None: # None is different from "", which is a valid value
repo_settings = git_provider.get_repo_settings() repo_settings = git_provider.get_repo_settings()
try: try: