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

add a constraint for websockets < 11.0 #2734

Closed
wants to merge 1 commit into from

Conversation

vcidst
Copy link

@vcidst vcidst commented Apr 4, 2023

Based on a discussion at #2733

@vcidst vcidst requested a review from a team as a code owner April 4, 2023 11:34
@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration .github/workflows/codeql-analysis.yml:analyze/language:python. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

@robd003
Copy link
Contributor

robd003 commented May 11, 2023

@ahopkins Can you please merge this?

@ahopkins
Copy link
Member

ahopkins commented Jul 5, 2023

@ahopkins Can you please merge this?

We need to get the tests passing first.

@vcidst vcidst closed this Oct 25, 2023
@Tronic
Copy link
Member

Tronic commented Oct 25, 2023

Has the problem been resolved? Sorry for delays with processing bugs / PRs. The core team has been quite overloaded lately.

@vcidst
Copy link
Author

vcidst commented Oct 25, 2023

Not really, I was cleaning up my old Pull Requests and noticed this had been open for some time with no activity. We've upgraded to a newer version of Sanic which doesn't have this problem

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.

None yet

4 participants