Bbrsofiane/maple upgrade#438
Conversation
BbrSofiane
force-pushed
the
bbrsofiane/maple-upgrade
branch
from
b6a1e3c to
24835ce
Compare
|
|
||
|
|
||
| CELERY_CHECK_MSG_PREFIX = 'figures-devsite-celery-check' | ||
| CELERY_CHECK_MSG_PREFIX = "figures-devsite-celery-check" |
There was a problem hiding this comment.
Is there a problem with the single quotes?
There was a problem hiding this comment.
No, I had black enabled at some point. I have disabled it since but looks like I missed some of the changes that were made.
There was a problem hiding this comment.
Thanks. Appreciate that!
|
@BbrSofiane Anything you want help with in the Maple upgrade? |
|
@johnbaldwin I could do with some pointers on how to configure some values like I will have more specific questions once I'm in front of the code. I have some time set aside Wednesday to work on this. |
…-merged John/develop to maple upgrade merged
OmarIthawi
left a comment
OmarIthawi
left a comment
There was a problem hiding this comment.
there's a git-merge leftover
Updatedg Tox to run for Maple only, removing prior Open edX release references, as the Figures Maple support release series makes a clean break from prior Open edX releases * The community tests now run * Also, pytest can run on its own without Tox * The pylint target also now runs and show lint errors * remarked out multisite mode because edx-organizations still requires update * Remarked out edx_lint_check. It fails locally (at least it has always failed locally for me)
Maple upgrade - Tox update + test pruning
3 participants
Change description
Type of change
Related issues
Checklists
Development
Security
Code review