Kyle McCormick
Kyle McCormick
I have updated the **Django settings** to clarify that we will also be removing the STATIC_ROOT_BASE Django setting.
Now that this is accepted, I'll split the removal into three steps: - [x] https://github.com/openedx/edx-platform/pull/34829 - [ ] https://github.com/openedx/edx-platform/pull/34830 - [ ] https://github.com/openedx/edx-platform/pull/34832 Step 1 is free of downstream contingencies....
That workflow update ^ is now handled. Here is an epic to handle the remaining upstream CI updates: * https://github.com/openedx/edx-platform/issues/34845
As of **May 7th 2024**, the operator-facing Paver commands have been fully replaced with npm & django commands. The replacement commands are being invoked in Redwood (directly) and on edx.org...
What @feanil says is right. I've tried to amend the description to be a bit clearer. In retrospect, it may have been clearer to just have two DEPR tickets, but...
Nice, I'll take a look soon.
Sorry, Ed. Busy on-call week. This is still on my radar.
Hah, I recall having this conversation at an edX Arch Lunch when I was an intern in 2017. Nobody could name a reason for keeping a separate FEATURES dict, especially...
I like `TOGGLE` because it clarifies that the variable a toggle, without the awkwardness of `ENABLE_BLAH.is_enabled()`
1. I perused [lms/envs/common.py](https://github.com/openedx/edx-platform/blob/master/lms/envs/common.py) and found perhaps 1/3 of the `ENABLE_` settings to default on, so I wouldn't consider that to be a naming convention worth preserving. 2. All it...