-
Notifications
You must be signed in to change notification settings - Fork 26
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.
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
[Discovery] Improve common constraints experience #355
Comments
Original comment from @timmc-edx: Robert and I discussed an alternative to the wget/sed approach today:
Effects:
|
The following is related to In #349 (review), a common constraint was removed once the openedx org was ready, but the edx org repos were not necessarily ready. The question is whether or not we need a capability for adding org specific common-constraints? |
Issue description and initial comments moved from private ticket: https://2u-internal.atlassian.net/browse/BOM-2721
The process around
common_constraints.txt
has some issues as we work through upgrades.constraints.txt
, which is where you might expect to do this when working through an upgrade that isn't yet available globally.See openedx/auth-backends#125 (comment) for some related discussion.
Additional Notes:
base.in
.base.in
, and not just rely oncommon_constraints.txt
.common_constraints.txt
, we either need to:common_constraints.txt
(if we are ready), orcommon_constraints.txt
for the library update, so no one will try to pick up the new version of the library which will conflict with the active common constraint on the dependency.The text was updated successfully, but these errors were encountered: