edx-platform
edx-platform copied to clipboard
Jdmulloy/fix import depr warning
Please consider the following when opening a pull request:
- Link to the relevant JIRA ticket(s) and tag any relevant team(s).
- Squash your changes down into one or more discrete commits. In each commit, include description that could help a developer several months from now.
- If running
make upgrade
, run as close to the time of merging as possible to avoid accidentally downgrading someone else's package. Put the output ofmake upgrade
in its own separate commit, decoupled from other code changes. - Aim for comprehensive test coverage, but remember that automated testing isn't a substitute for manual verification.
- Carefully consider naming, code organization, dependencies when adding new code. Code that is amenable to refactoring and improvement benefits all platform developers, especially given the size and scope of edx-platform. Consult existing Architectural Decision Records (ADRs), including those concerning the app(s) you are changing and those concerning edx-platform as a whole.
Your PR has finished running tests. The following contexts failed:
- jenkins/a11y
- jenkins/bokchoy
- jenkins/python
- jenkins/quality
📣 💥 Heads-up: You must either rebase onto master or merge master into your branch to avoid breaking the build.
We recently removed diff-quality and introduced lint-amnesty. This means that the automated quality check that has run on your branch doesn't work the same way it will on master. If you have introduced any quality failures, they might pass on the PR but then break the build on master.
This branch has been detected to not have commit 2e335653 as an ancestor. Here's how to see for yourself:
git merge-base --is-ancestor 2e335653 jdmulloy/fix_import_depr_warning && echo "You're all set" || echo "Please rebase onto master or merge master to your branch"
If you have any questions, please reach out to the Architecture team (either #edx-shared-architecture on Open edX Slack or #architecture on edX internal).