search

openedx-unsupported / edx-analytics-pipeline

GNU Affero General Public License v3.0
91 stars 116 forks source link

remove OEP-10 so this repo is no longer shipped with openedx releases. #841

Closed macdiesel closed 10 months ago

macdiesel commented 4 years ago

Analytics Pipeline Pull Request

Make sure that the following steps are done before merging:

natabene commented 4 years ago

@macdiesel I am for it, but cannot really review from engineering perspective. @nedbat , any comments?

codecov[bot] commented 4 years ago

Codecov Report

Merging #841 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #841   +/-   ##
=======================================
  Coverage   74.01%   74.01%           
=======================================
  Files         208      208           
  Lines       23861    23861           
=======================================
  Hits        17661    17661           
  Misses       6200     6200

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a58bd93...8a52a86. Read the comment docs.

kdmccormick commented 10 months ago

Relevant DEPR: https://github.com/openedx/public-engineering/issues/36

EDIT: That DEPR is just for one view, not the whole system.