refactor!: stop using import_shims
Attempting to import packages from lms/djangoapps, cms/djangoapps, or common/djangoapps as if they are import roots will now simply raise ImportErrors (like any other invalid import) instead of DeprecatedEdxPlatformImportError. See docs/decisions/0007-sys-path-modification-removal.rst for more details.
This commit is contained in:
committed by
Kyle McCormick
parent
f92740b39f
commit
4531e15519
@@ -492,11 +492,6 @@ COURSES_ROOT = ENV_ROOT / "data"
|
||||
|
||||
GITHUB_REPO_ROOT = ENV_ROOT / "data"
|
||||
|
||||
# TODO: This path modification exists as temporary support for deprecated import patterns.
|
||||
# It will be removed in an upcoming Open edX release.
|
||||
# See docs/decisions/0007-sys-path-modification-removal.rst
|
||||
sys.path.append(REPO_ROOT / 'import_shims' / 'studio')
|
||||
|
||||
# For geolocation ip database
|
||||
GEOIP_PATH = REPO_ROOT / "common/static/data/geoip/GeoLite2-Country.mmdb"
|
||||
|
||||
|
||||
@@ -970,11 +970,6 @@ NODE_MODULES_ROOT = REPO_ROOT / "node_modules"
|
||||
|
||||
DATA_DIR = COURSES_ROOT
|
||||
|
||||
# TODO: This path modification exists as temporary support for deprecated import patterns.
|
||||
# It will be removed in an upcoming Open edX release.
|
||||
# See docs/decisions/0007-sys-path-modification-removal.rst
|
||||
sys.path.append(REPO_ROOT / 'import_shims' / 'lms')
|
||||
|
||||
# For Node.js
|
||||
|
||||
system_node_path = os.environ.get("NODE_PATH", NODE_MODULES_ROOT)
|
||||
|
||||
@@ -30,7 +30,6 @@ SKIP_DIRS = (
|
||||
'spec',
|
||||
'test_root',
|
||||
'vendor',
|
||||
'import_shims',
|
||||
)
|
||||
|
||||
|
||||
|
||||
Reference in New Issue
Block a user