Skip to content

Conversation

chintanjoshi-apphelix-2u
Copy link
Contributor

Description

This reverts PR #230

Earlier PR Added changes for adding NPM aliases, it was merged in hopes of fixing the pipeline failure which is

Traceback (most recent call last):
  File "/edx/app/edxapp/edx-platform/scripts/compile_sass.py", line 451, in <module>
    main(prog_name="npm run compile-sass --")
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/core.py", line 1442, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/core.py", line 1363, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/core.py", line 1226, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/core.py", line 794, in invoke
    return callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/decorators.py", line 34, in new_func
    return f(get_current_context(), *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/edx-platform/scripts/compile_sass.py", line 373, in main
    compile_sass_dir(
  File "/edx/app/edxapp/edx-platform/scripts/compile_sass.py", line 252, in compile_sass_dir
    raise Exception(f"Failed to compile {source}: {output_text}")
Exception: Failed to compile lms/static/sass/lms-mobile-rtl.scss: Error: File to import not found or unreadable: brand-edx.org/paragon/fonts
       Parent style sheet: /edx/var/edx-themes/edx-themes/edx-platform/edx.org-next/lms/static/sass/partials/lms/theme/_variables.scss
        on line 4 of ../../../var/edx-themes/edx-themes/edx-platform/edx.org-next/lms/static/sass/partials/lms/theme/_variables.scss
>> @import 'brand-edx.org/paragon/fonts';

@chintanjoshi-apphelix-2u chintanjoshi-apphelix-2u force-pushed the chintanjoshi-apphelix-2u/fix-npm-tasks branch from 784e5c5 to ace832f Compare August 19, 2025 08:20
@chintanjoshi-apphelix-2u chintanjoshi-apphelix-2u marked this pull request as ready for review August 19, 2025 08:20
@UsamaSadiq UsamaSadiq merged commit 61c67d6 into master Aug 19, 2025
3 checks passed
@UsamaSadiq UsamaSadiq deleted the chintanjoshi-apphelix-2u/fix-npm-tasks branch August 19, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants