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

feat: [ACI-1031] Upgrade requirements to upstream version #181

Merged

Conversation

KyryloKireiev
Copy link

Description
Upgrade requirements to current upstream's versions.

YouTrack
https://youtrack.raccoongang.com/issue/ACI-1031/Review-PRs-comments

@KyryloKireiev KyryloKireiev requested a review from wowkalucky July 16, 2024 12:46
@KyryloKireiev KyryloKireiev self-assigned this Jul 16, 2024
@@ -304,7 +314,7 @@ edx-event-bus-kafka==5.7.0
# via
# -r requirements/dev.txt
# -r requirements/production.txt
edx-event-bus-redis @ git+https://github.com/openedx/event-bus-redis.git@v0.5.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

This downgrade is strange.

Copy link
Author

Choose a reason for hiding this comment

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

Fixed

@@ -3,6 +3,16 @@
# See BOM-2721 for more details.
# Below is the copied and edited version of common_constraints

# This is a temporary solution to override the real common_constraints.txt
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pls remove this diff manually.

Copy link
Author

Choose a reason for hiding this comment

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

Removed

@wowkalucky wowkalucky merged commit 7b75760 into aci.upstream Jul 16, 2024
2 checks passed
@wowkalucky wowkalucky deleted the kireiev/ACI-1031/feat/upgrade_requirements_v2 branch July 16, 2024 13:58
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.

2 participants