Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Process edx-platform assets without Paver #31798

Closed
Tracked by #146
kdmccormick opened this issue Feb 21, 2023 · 0 comments
Closed
Tracked by #146

Process edx-platform assets without Paver #31798

kdmccormick opened this issue Feb 21, 2023 · 0 comments
Assignees
@kdmccormick
Labels
duplicate This issue or pull request already exists elsewhere epic Large unit of work, consisting of multiple tasks

Comments

@kdmccormick
Copy link
Member

kdmccormick commented Feb 21, 2023 •
edited
Loading

Dupe of #31895, #31895
This was referenced Feb 21, 2023
Closed
Closed
Closed
Closed
Open
Closed
Closed
Merged
Merged
kdmccormick added a commit that referenced this issue Feb 22, 2023
@kdmccormick
refactor!: remove unused update-assets-dev.sh (#31810)
b140b7e 
The file was introduced as part of the "decentralized devstack" spike.
We wanted a working Dockerfile in the edx-platform root,
and we wanted to speed it up by not relying on Paver.

edx-platform now has a working Dockerfile, and it does not use this script.

The paver-free-assets initiative is being continued here:
#31798
It will result in a new, prod-ready asset building script.
This was referenced Feb 24, 2023
Closed
Open
@kdmccormick kdmccormick self-assigned this Feb 24, 2023
This was referenced Mar 7, 2023
Closed
Closed
Closed
@kdmccormick kdmccormick changed the title Process edx-platform assets without Paver Paver-free assets Mar 19, 2023
@kdmccormick kdmccormick changed the title Paver-free assets Process edx-platform assets without Paver Mar 19, 2023
@kdmccormick kdmccormick moved this from 📥 New to 📋 Refined in Developer Experience Working Group Mar 19, 2023
kdmccormick added a commit to kdmccormick/tutor that referenced this issue Apr 14, 2023
@kdmccormick
feat: add EDX_PLATFORM_NEW_ASSET_PROCESSING flag
43b7891 
When enabled, this flag tells Tutor's openedx-assets script
(which processes assets using Python and edx-platform's Paver tools)
to instead proxy through to edx-platform's new asset processing system
(`scripts/build-assets.sh` and `./manage.py collectstatic`).

Also, when enabled, openedx-assets will print out the shell command
that could have been used to directly invoke the new asset processing
system. In the future, openedx-assets will be deprecated, and the
direct shell command will be the canonical way of compiling assets.

Part of: openedx/edx-platform#31798
ADR: https://github.com/openedx/edx-platform/blob/master/docs/decisions/0017-reimplement-asset-processing.rst
@kdmccormick kdmccormick added the epic Large unit of work, consisting of multiple tasks label Apr 27, 2023
@kdmccormick kdmccormick moved this to 🛠️ In Progress in Developer Experience Working Group Apr 27, 2023
@kdmccormick kdmccormick mentioned this issue Jul 11, 2023
Merged
@kdmccormick kdmccormick added the duplicate This issue or pull request already exists elsewhere label Mar 25, 2024
@kdmccormick kdmccormick closed this as not planned Won't fix, can't repro, duplicate, stale Mar 25, 2024
@github-project-automation github-project-automation bot moved this to Done in Arch-BOM Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kdmccormick kdmccormick

Labels
duplicate This issue or pull request already exists elsewhere epic Large unit of work, consisting of multiple tasks
Projects
No open projects
Developer Experience Working Group
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kdmccormick