Skip to content

Conversation

@edx-requirements-bot
Copy link
Contributor

Updated repo with setup.py changes for package extras, for more context please have a look into this issue

@angonz
Copy link
Contributor

angonz commented May 13, 2025

Hi @zubairshakoorarbisoft,
You were asked to review this PR. Please let me know if you can complete this review in order to merge this PR.

@UsamaSadiq
Copy link
Member

Hi @angonz
Zubair was in my team but he left the organization few months ago. This PR is just waiting for the owning team to be merged so you can go ahead and merge this.
For reference/context, similar change has already been merged across almost all of the openedx packages but this one got left out waiting on the owning/maintainer team.

@UsamaSadiq UsamaSadiq removed the request for review from zubairshakoorarbisoft May 13, 2025 15:01
@angonz
Copy link
Contributor

angonz commented May 13, 2025

Thanks @UsamaSadiq! I will go ahead and merge this.

@angonz angonz merged commit 72381e3 into master May 13, 2025
14 checks passed
@angonz angonz deleted the jenkins/zshkoor/setup-py-updated-cc00c32 branch May 13, 2025 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants