Skip to content

Commit

Permalink
build: compile/watch sass with new npm scripts
Browse files Browse the repository at this point in the history
`paver` commands are deprecated for managing static assets. Starting in
Sumac, only `npm run` commands will be supported for managing static
assets.

To ease the transition, both `paver` and `npm run` commands will work in
Redwood. However, we want to stop using the *implementations* of the
`paver` asset commands right now, as they are blocking the Python 3.11
upgrade. This will also make the removal of `paver` commands more
straightforward come Sumac.

So, this commit turns these commands/functions:
* paver compile_sass (used by configuration)
* paver watch_sass (used by configuration and devstack)
* pavelib/assets.py:_compile_sass (used by Tutor)

into very thin wrappers around the new `npm run` commands. Each of these
paver routines now raise a loud deprecation warning, including a message
of the `npm run` command that the operator can switch to.
We expect no impact to site operators or end users.

openedx#31895
  • Loading branch information
kdmccormick authored and KyryloKireiev committed Apr 24, 2024
1 parent 6387b8d commit ac6c929
Show file tree
Hide file tree
Showing 3 changed files with 122 additions and 627 deletions.
75 changes: 0 additions & 75 deletions openedx/core/djangoapps/theming/paver_helpers.py

This file was deleted.

Loading

0 comments on commit ac6c929

Please sign in to comment.