search

openedx / edx-platform

The Open edX LMS & Studio, powering education sites around the world!
https://openedx.org
GNU Affero General Public License v3.0
7.32k stars 3.84k forks source link

Run depsplorer on edx-platform #32604

Open dianakhuang opened 1 year ago

dianakhuang commented 1 year ago

Run https://github.com/Nathan-Furnal/depslorer on edx-platform to see if there are any interesting dependencies we don't need anymore.

Acceptance Criteria

Anakin100100 commented 1 year ago

I can try to work on this

LakshyaKhatri commented 1 year ago

Hi @dianakhuang! I gave it a try and I was able to find out the unused dependencies using desplorer. I've one question around this before I start digging deeper into checking which of the dependencies are actually not being used:

Right now, I'm only looking at the auto-generated *.txt files for deprecated packages, and then I'll use them as a reference to remove the unused dependencies in the *.in files. Am I heading in the right direction?

LakshyaKhatri commented 1 year ago

Here the output from desplorer: Desplorer Output.md

Note that this is just the raw output, a lot of the dependencies are coming from inherited parent requirement files. Also, some of the .in files don't have a corresponding .txt files. I've to check them. Although, all the requirement files look pretty well documented, hope that will help me to navigate around easily.

I'll post the update soon. 😄

LakshyaKhatri commented 12 months ago

Hi @dianakhuang! Attaching the detailed output of each package in the edx-platform repo: Unused packages.pdf

The sole package deserving a consideration for deprecation is packaging. I was planning to share my findings during the last DEPR meeting, but it appears the meeting may have been canceled?

Please advise on the next steps to proceed.

dianakhuang commented 12 months ago

@LakshyaKhatri thank you! The DEPR meetings should still be going on. You should be find the calendar invite on this google calendar: https://calendar.google.com/calendar/u/0?cid=Y192ODZzaHJuZWdzaHNxZ3A0Zmoyazk0dTdiY0Bncm91cC5jYWxlbmRhci5nb29nbGUuY29t

We are meeting next Thursday.

LakshyaKhatri commented 12 months ago

Thanks so much @dianakhuang. I will join this Thursday 😄 🙌🏽

LakshyaKhatri commented 11 months ago

Hello again @dianakhuang, 👋🏽

just confirming once: as per the discussion we are going to remove the following dependencies:

correct?

dianakhuang commented 11 months ago

@LakshyaKhatri yup!

Abdujabbar commented 1 month ago

@dianakhuang

Can I move forward with the current issue? Are you ok with that @LakshyaKhatri? If you don't have a time?

LakshyaKhatri commented 1 month ago

@Abdujabbar yes, please go ahead 👍🏽