openedx-unsupported / wg-developer-experience

Old issue repository for the former Developer Experience Working Group
4 stars 0 forks source link

Install all edx-platform dependencies from PyPI, not GitHub #153

Closed kdmccormick closed 1 year ago

kdmccormick commented 2 years ago

Background

Several edx-platform packages are installed directly from GitHub instead of PyPI. See: edx-platform's github.in. This is non-ideal because:

Acceptance Criteria

In edx-platform, remove all packages from github.in.

Investigate use of GitHub-hosted deps in other IDAs. Remove them, or create follow-up tickets.

Add guidance that steers developers away from using GitHub-hosted deps and/or makes it easier for them to publish their packages to PyPI. Consider using a repo health check or CODEOWNERS to help us hold ourselves to this.

### Tasks
   - [x] https://github.com/openedx/open-edx-proposals/pull/390
   - [x] https://github.com/openedx/edx-platform/pull/30719
   - [x] https://github.com/openedx/edx-platform/pull/31104
   - [x] https://github.com/openedx/open-edx-proposals/pull/450
   - [x] https://github.com/openedx/edx-platform/pull/31824
   - [x] https://github.com/openedx/codejail/pull/140
   - [x] https://github.com/openedx/codejail/pull/151
   - [x] https://github.com/openedx/edx-platform/pull/31683
   - [x] https://github.com/openedx/RateXBlock/issues/45
   - [x] https://github.com/openedx/xblock-drag-and-drop-v2/issues/299
   - [x] https://github.com/openedx/blockstore/pull/202
   - [x] https://github.com/openedx/edx-platform/pull/30601
   - [x] https://github.com/openedx/edx-platform/pull/31236
   - [x] https://github.com/openedx/blockstore/issues/212
   - [x] https://github.com/openedx/edx-platform/pull/31271
   - [x] https://github.com/openedx/MongoDBProxy/issues/18
   - [ ] https://github.com/openedx/edx-platform/pull/32304
   - [x] https://github.com/openedx/olxcleaner/pull/16
   - [x] https://github.com/openedx/olxcleaner/pull/17
   - [x] https://github.com/openedx/edx-platform/pull/31819
   - [x] https://github.com/openedx/edx-platform/pull/31279
   - [x] https://github.com/open-craft/xblock-poll/issues/98
   - [x] https://github.com/openedx/edx-platform/pull/31718
   - [x] https://github.com/openedx/edx-platform/pull/31201
kdmccormick commented 1 year ago

@jmbowman I'm trying to collect all work around removing GitHub-hosted dependencies on this issue. Does BOM have any planned work around this and/or tickets that would be good for me to link from here?

jmbowman commented 1 year ago

There's one for codejail that hasn't been copied from Jira to a GitHub Issue yet, but I think that's it for now. I'll get some more tickets for this added to the Arbi-BOM board.

aht007 commented 1 year ago

xblock-poll is owned by opencraft. I have made an issue in xblock-poll repository to release the package to PyPI.

kdmccormick commented 1 year ago

Thanks for continuing to chip away at this @aht007 . Let me know if you or your team needs any help.

UsamaSadiq commented 1 year ago

Created PR to move olxcleaner from GitHub to PyPI. It'll be merged after completely testing the change using sandbox.

kdmccormick commented 1 year ago

As of today, github.in is empty!