edx / edx-arch-experiments

A plugin to include applications under development by the architecture team at edx
GNU Affero General Public License v3.0
0 stars 3 forks source link

Reduce friction for adding temporary code to edx-platform #206

Open robrap opened 1 year ago

robrap commented 1 year ago

AC:

Notes/Questions:

robrap commented 1 year ago

Additional notes:

  1. Andy S. experimented with this process. See private Slack thread: https://twou.slack.com/archives/C04ACDVM6A1/p1678288563912389. We can review what went well or not.
  2. We learned that arch-bom was required to merge PRs to the edx-arch-experiments project. @rgraber proposed that this should be left-as is, until we can lint for a "remove-by date", so the implementer is aware they must clean-up by a certain date. I'm not sure when and if this will be worth while implementing, but we could add to our docs on this that teams can't merge without arch-bom approval, but that the only thing we plan to review is whether or not there is a "remove-by date."
robrap commented 1 year ago

Relates to https://github.com/edx/edx-arch-experiments/issues/422 for improving how we manage private dependencies.