fix: replace isort pragmas with ruff-equivalent noqa suppressions

isort:skip_file and isort:skip pragmas are not respected by ruff.
Replace them with ruff equivalents:
- File-level: # ruff: noqa: I001 (cms/__init__.py, cms/wsgi.py,
  openedx/core/lib/safe_lxml/etree.py, cms/conftest.py)
- Line-level: # noqa: I001 (manage.py, cms/urls.py)

We also added some suppressions to settings file where the wildcard
import and other import ordering is important.
Co-Authored-By: Claude Sonnet 4.6 <[email protected]>

Author: feanil

cms/__init__.py

@@ -1,9 +1,9 @@
+# ruff: noqa: I001
 """
 Celery needs to be loaded when the cms modules are so that task
 registration and discovery can work correctly.
 
 Import sorting is intentionally disabled in this module.
-isort:skip_file
 """
 
 

cms/conftest.py

@@ -1,3 +1,4 @@
+# ruff: noqa: I001
 """
 Studio unit test configuration and fixtures.
 
@@ -13,7 +14,7 @@
 from openedx.core.pytest_hooks import DeferPlugin
 
 # Patch the xml libs before anything else.
-from openedx.core.lib.safe_lxml import defuse_xml_libs  # isort:skip
+from openedx.core.lib.safe_lxml import defuse_xml_libs  # must patch xml libs before other imports execute
 defuse_xml_libs()
 
 

cms/envs/devstack.py

@@ -1,3 +1,4 @@
+# ruff: noqa: I001 - settings file: star-import order is semantically significant
 """
 Specific overrides to the base prod settings to make development easier.
 """

cms/envs/docker-production.py

@@ -1,3 +1,4 @@
+# ruff: noqa: I001 - settings file: star-import order is semantically significant
 """
 Specific overrides to the base prod settings for a docker production deployment.
 """

cms/envs/production.py

@@ -1,3 +1,4 @@
+# ruff: noqa: I001 - settings file: star-import order is semantically significant
 """
 Override common.py with key-value pairs from YAML (plus some extra defaults & post-processing).
 

cms/envs/test.py

@@ -13,7 +13,8 @@
 # pylint: disable=wildcard-import, unused-wildcard-import
 
 
-import os
+
+import os  # noqa: I001 - suppresses linting for this whole block, sort imports manually as needed
 import tempfile
 
 from django.utils.translation import gettext_lazy
@@ -22,10 +23,10 @@
 from openedx.core.djangoapps.plugins.constants import ProjectType, SettingsType
 from openedx.core.lib.derived import derive_settings
 from openedx.core.lib.features_setting_proxy import FeaturesProxy
-from openedx.envs.test import *  # pylint: disable=wrong-import-order
 from xmodule.modulestore.modulestore_settings import update_module_store_settings  # pylint: disable=wrong-import-order
 
 from .common import *
+from openedx.envs.test import *  # must come after .common to override Derived values with literals
 
 # A proxy for feature flags stored in the settings namespace
 FEATURES = FeaturesProxy(globals())

cms/urls.py

@@ -263,7 +263,7 @@
 
 # Enable Web/HTML Certificates
 if settings.FEATURES.get('CERTIFICATES_HTML_VIEW'):
-    from cms.djangoapps.contentstore.views.certificates import (
+    from cms.djangoapps.contentstore.views.certificates import (  # noqa: I001 - conditional import inside if block
         CertificateActivationAPIView,
         CertificateDetailAPIView,
         certificates_list_handler,
@@ -342,9 +342,9 @@
     ]
 
 # pylint: disable=wrong-import-position, wrong-import-order
-from edx_django_utils.plugins import get_plugin_url_patterns  # isort:skip
+from edx_django_utils.plugins import get_plugin_url_patterns  # noqa: I001 - must be after urlpatterns are built
 # pylint: disable=wrong-import-position
-from openedx.core.djangoapps.plugins.constants import ProjectType  # isort:skip
+from openedx.core.djangoapps.plugins.constants import ProjectType  # noqa: I001 - must be after urlpatterns are built
 
 urlpatterns.extend(get_plugin_url_patterns(ProjectType.CMS))
 

cms/wsgi.py

@@ -1,3 +1,4 @@
+# ruff: noqa: I001
 """
 WSGI config for CMS.
 
@@ -8,7 +9,6 @@
 ``WSGI_APPLICATION`` setting.
 
 Import sorting is intentionally disabled in this module.
-isort:skip_file
 """
 
 # Patch the xml libs before anything else.

lms/envs/common.py

@@ -44,6 +44,7 @@
 
 from corsheaders.defaults import default_headers as corsheaders_default_headers
 from django.utils.translation import gettext_lazy as _
+from edx_django_utils.plugins import add_plugins, get_plugin_apps
 from enterprise.constants import (
     DEFAULT_ENTERPRISE_ENROLLMENT_INTENTIONS_ROLE,
     ENTERPRISE_ADMIN_ROLE,
@@ -57,11 +58,12 @@
     ENTERPRISE_SSO_ORCHESTRATOR_OPERATOR_ROLE,
     PROVISIONING_ENTERPRISE_CUSTOMER_ADMIN_ROLE,
     PROVISIONING_PENDING_ENTERPRISE_CUSTOMER_ADMIN_ROLE,
-    SYSTEM_ENTERPRISE_PROVISIONING_ADMIN_ROLE
+    SYSTEM_ENTERPRISE_PROVISIONING_ADMIN_ROLE,
 )
 from openedx_content.settings_api import openedx_content_backcompat_apps_to_install
 from path import Path as path
 
+from openedx.core.djangoapps.plugins.constants import ProjectType, SettingsType
 from openedx.core.lib.derived import Derived
 from openedx.core.lib.features_setting_proxy import FeaturesProxy
 from openedx.envs.common import *  # pylint: disable=wildcard-import
@@ -2953,13 +2955,6 @@
 
 ############## Plugin Django Apps #########################
 
-from edx_django_utils.plugins import (  # pylint: disable=wrong-import-position,wrong-import-order
-    add_plugins,
-    get_plugin_apps
-)
-
-from openedx.core.djangoapps.plugins.constants import ProjectType, SettingsType  # pylint: disable=wrong-import-position
-
 INSTALLED_APPS.extend(get_plugin_apps(ProjectType.LMS))
 add_plugins(__name__, ProjectType.LMS, SettingsType.COMMON)
 

lms/envs/devstack.py

@@ -1,3 +1,4 @@
+# ruff: noqa: I001 - settings file: star-import order is semantically significant
 """
 Specific overrides to the base prod settings to make development easier.
 """