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.

Sign up for GitHub

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

Jump to bottom

pin tenacity to pre 8.4.0 #41

Closed
wants to merge 2 commits into from
Closed

pin tenacity to pre 8.4.0 #41

wants to merge 2 commits into from

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented Jun 17, 2024

@evgeni
Copy link
Member Author

evgeni commented Jun 17, 2024

A fix is being worked on:

ekohl
ekohl requested changes Jun 17, 2024
View reviewed changes
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

I'd also be fine with waiting a little bit to see if a fix is released.

requirements.txt Outdated Show resolved Hide resolved
@evgeni @ekohl
Update requirements.txt
d90d137 
Co-authored-by: Ewoud Kohl van Wijngaarden <ewoud@kohlvanwijngaarden.nl>
@evgeni
Copy link
Member Author

evgeni commented Jun 18, 2024

8.4.1 was released and things are working again. Shall we still merge this (8.4.0 wasn't yanked), or ignore it?

@ekohl
Copy link
Member

ekohl commented Jun 18, 2024

Let's ignore it

@evgeni evgeni closed this Jun 18, 2024
@evgeni evgeni deleted the pin-tenacity branch June 18, 2024 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ekohl ekohl ekohl requested changes

Assignees
No one assigned
Labels
Waiting on contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@evgeni @ekohl