Skip to content

Conversation

rjnrohit
Copy link
Contributor

@rjnrohit rjnrohit commented Feb 6, 2025

Remove the check if the number of the requests are more than 3

Copy link
Contributor

@MarekM25 MarekM25 left a comment

Choose a reason for hiding this comment

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

LGTM, but for future try to cover such checks with tests

@rjnrohit rjnrohit merged commit f692ff1 into master Feb 6, 2025
79 checks passed
@rjnrohit rjnrohit deleted the requests_type_check branch February 6, 2025 13:03
@rjnrohit
Copy link
Contributor Author

rjnrohit commented Feb 6, 2025

LGTM, but for future try to cover such checks with tests

okay

kamilchodola pushed a commit that referenced this pull request Feb 8, 2025
kamilchodola added a commit that referenced this pull request Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants