Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: setup.py update using script #116

Merged
merged 3 commits into from
Feb 26, 2024

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

@felipemontoya
Copy link
Member

I will wait for the sample PR openedx/repo-tools#436 to be accepted and merged before reviewing

@mariajgrimaldi
Copy link
Member

Hi there @zubairshakoorarbisoft! Can can we do to move forward with your PR in the repo-tools repo?

Copy link
Member

@mariajgrimaldi mariajgrimaldi left a comment

Choose a reason for hiding this comment

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

We already merged these changes in openedx/openedx-events@650ec00, so I'll go ahead and merge them here

@mariajgrimaldi mariajgrimaldi merged commit 798d1fb into main Feb 26, 2024
8 checks passed
@mariajgrimaldi mariajgrimaldi deleted the jenkins/zshkoor/setup-py-updated-241d282 branch February 26, 2024 14:08
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.

3 participants