-
Notifications
You must be signed in to change notification settings - Fork 421
Feat:Auto-sync dependencies between code and virtual env for lms #1068
Conversation
Thanks for the pull request, @XLAR-8! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
Hi @XLAR-8, thank you for this contribution! As the bot mentioned, the next step will be for you to sign the Open edX Contributor Agreement. Then we can enable tests for this PR, and get it into the review queue. |
@itsjeyd I have already sign the Open edX Contributor Agreement. |
Hey @XLAR-8, now that the CLA check passed, could you please rebase your branch on latest master (or merge the latest commits)? Once that's done we'll kick off a test run. |
Updating via a merge commit, which will re-run the tests. |
@XLAR-8 looks like several of the provisions are failing with this change, can you have a look? |
@XLAR-8 A friendly reminder to have a look at the tests that are failing here. Once everything is green we'll line up your changes for engineering review. |
Hi @XLAR-8, just checking in to see if we should keep this PR open? |
@XLAR-8 Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future. |
Solves issue: devstack/issues/1028
description:
Auto-sync dependencies between code and virtual env for LMS for now, can be extended for other services too.
I've completed each of the following or determined they are not applicable: