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(typing): Fix typing in websocket listener class #3765

Merged
merged 1 commit into from
Nov 17, 2024

Conversation

olzhasar
Copy link
Contributor

Description

Fixes #3763

It seems that declaring the actual method is the only way to comply with mypy in such cases.
There is a ruff rule that forbids empty methods in abstract classes, had to disable that for the relevant module.

Closes

@olzhasar olzhasar requested review from a team as code owners September 29, 2024 12:43
@github-actions github-actions bot added area/dependencies This PR involves changes to the dependencies size: small type/bug pr/external Triage Required 🏥 This requires triage labels Sep 29, 2024
Copy link

codecov bot commented Sep 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.41%. Comparing base (1b2ef9a) to head (b206ca9).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3765   +/-   ##
=======================================
  Coverage   98.41%   98.41%           
=======================================
  Files         339      339           
  Lines       15371    15371           
  Branches     1694     1694           
=======================================
  Hits        15128    15128           
  Misses        114      114           
  Partials      129      129           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@olzhasar
Copy link
Contributor Author

Is there anything preventing this PR from being merged?

@cofin cofin force-pushed the websockets-type-fix branch from 4530c6e to b206ca9 Compare November 17, 2024 21:48
@cofin cofin enabled auto-merge (squash) November 17, 2024 21:50
@cofin cofin disabled auto-merge November 17, 2024 21:51
@cofin
Copy link
Member

cofin commented Nov 17, 2024

@all-contributors add @olzhasar for bugs and docs

Copy link
Contributor

@cofin

I've put up a pull request to add @olzhasar! 🎉

@cofin cofin enabled auto-merge (squash) November 17, 2024 21:52
@cofin cofin merged commit a3631e2 into litestar-org:main Nov 17, 2024
24 checks passed
Copy link

Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/3765

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies This PR involves changes to the dependencies pr/external pr/internal size: small Triage Required 🏥 This requires triage type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Incompatible signatures when using WebsocketListener class
2 participants