From d76bed4bbba12c6d05ad5315ef58ad823d4a393c Mon Sep 17 00:00:00 2001
From: Feanil Patel <feanil@axim.org>
Date: Thu, 30 Oct 2025 10:55:38 -0400
Subject: [PATCH] style: Drop unused imports. (#37572)

* style: Drop unused imports.

I was looking at these files for some other reason and saw that these
imports were unused. Some of it looks like an artifact of the hoisting
of some settings up to the `openedx/env/common.py` file and others look
like some ancient unused imports.


Co-authored-by: Kyle McCormick <kyle@axim.org>
---
 cms/envs/common.py     | 11 -----------
 lms/envs/common.py     |  2 --
 openedx/envs/common.py |  2 ++
 3 files changed, 2 insertions(+), 13 deletions(-)

diff --git a/cms/envs/common.py b/cms/envs/common.py
index a40ef75dd3..d4fb098f6f 100644
--- a/cms/envs/common.py
+++ b/cms/envs/common.py
@@ -39,9 +39,7 @@ When refering to XBlocks, we use the entry-point name. For example,
 # pylint: disable=unused-import, useless-suppression, wrong-import-order, wrong-import-position
 
 import importlib.util
-import json
 import os
-import sys
 
 from corsheaders.defaults import default_headers as corsheaders_default_headers
 from datetime import timedelta
@@ -53,19 +51,12 @@ import lms.envs.common
 from openedx.envs.common import *  # pylint: disable=wildcard-import
 
 from path import Path as path
-from django.urls import reverse_lazy
 
 from lms.djangoapps.lms_xblock.mixin import LmsBlockMixin
 from cms.lib.xblock.authoring_mixin import AuthoringMixin
 from cms.lib.xblock.upstream_sync import UpstreamSyncMixin
 from xmodule.modulestore.edit_info import EditInfoMixin
-from openedx.core.djangoapps.theming.helpers_dirs import (
-    get_themes_unchecked,
-    get_theme_base_dirs_from_settings
-)
-from openedx.core.lib.license import LicenseMixin
 from openedx.core.lib.derived import Derived
-from openedx.core.release import doc_version
 from openedx.core.lib.features_setting_proxy import FeaturesProxy
 
 # A proxy for feature flags stored in the settings namespace
@@ -735,8 +726,6 @@ STATICFILES_FINDERS = [
     'pipeline.finders.PipelineFinder',
 ]
 
-from openedx.core.lib.rooted_paths import rooted_glob
-
 PIPELINE['STYLESHEETS'] = {
     'style-vendor': {
         'source_filenames': [
diff --git a/lms/envs/common.py b/lms/envs/common.py
index 3dde7156b9..b28c9099e9 100644
--- a/lms/envs/common.py
+++ b/lms/envs/common.py
@@ -41,10 +41,8 @@ Conventions
 # pylint: disable=invalid-name
 
 import importlib.util
-import sys
 import os
 
-import django
 from corsheaders.defaults import default_headers as corsheaders_default_headers
 from path import Path as path
 from django.utils.translation import gettext_lazy as _
diff --git a/openedx/envs/common.py b/openedx/envs/common.py
index ada6fd3f0b..c00a89cf4f 100644
--- a/openedx/envs/common.py
+++ b/openedx/envs/common.py
@@ -37,6 +37,8 @@ from openedx.core.djangoapps.theming.helpers_dirs import (
     get_theme_base_dirs_from_settings
 )
 
+# We have legacy components that reference these constants via the settings module.
+# New code should import them directly from `openedx.core.constants` instead.
 from openedx.core.constants import (  # pylint: disable=unused-import
     ASSET_KEY_PATTERN,
     COURSE_KEY_REGEX,