Skip to content

chore: use cls for classmethods#545

Merged
dlech merged 1 commit intomainfrom
use-cls
Nov 16, 2025
Merged

chore: use cls for classmethods#545
dlech merged 1 commit intomainfrom
use-cls

Conversation

@dlech
Copy link
Member

@dlech dlech commented Nov 16, 2025

Replace uses of klass with the expected cls in the one and only classmethod definition.

Replace uses of `klass` with the expected `cls` in the one and only
classmethod definition.
@codecov
Copy link

codecov bot commented Nov 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.24%. Comparing base (ecf5f45) to head (84f8e17).
⚠️ Report is 36 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #545      +/-   ##
==========================================
+ Coverage   85.18%   85.24%   +0.05%     
==========================================
  Files          28       28              
  Lines        3416     3416              
  Branches      600      600              
==========================================
+ Hits         2910     2912       +2     
+ Misses        313      312       -1     
+ Partials      193      192       -1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed-hq
Copy link

codspeed-hq bot commented Nov 16, 2025

CodSpeed Performance Report

Merging #545 will not alter performance

Comparing use-cls (84f8e17) with main (ecf5f45)

Summary

✅ 6 untouched

@dlech dlech merged commit d87af15 into main Nov 16, 2025
24 checks passed
@dlech dlech deleted the use-cls branch November 16, 2025 22:01
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

Comments