Skip to content
This repository has been archived by the owner on Oct 5, 2023. It is now read-only.

fix: setup.py update using script #226

Closed

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

Copy link
Contributor

@feanil feanil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like this has added a lot more than just the setup.py changes. Is this expected? Also, the new content seems to have various mixed copyright statements in it.

@feanil
Copy link
Contributor

feanil commented Sep 15, 2023

@zubairshakoorarbisoft can you explain the approval, this seems very incorrect especially with the additional content files.

@zubairshakoorarbisoft zubairshakoorarbisoft dismissed their stale review September 15, 2023 17:44

Approved mistakenly.

@farhan
Copy link
Contributor

farhan commented Sep 26, 2023

Closing as we have deprecated the repo:
#231

@farhan farhan closed this Sep 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants