Skip to content

Update debug_assert which pointed at missing method#21969

Merged
charliermarsh merged 1 commit intoastral-sh:mainfrom
PeterJCLaw:fix-debug-assert-message
Dec 13, 2025
Merged

Update debug_assert which pointed at missing method#21969
charliermarsh merged 1 commit intoastral-sh:mainfrom
PeterJCLaw:fix-debug-assert-message

Conversation

@PeterJCLaw
Copy link
Contributor

Summary

I assume that the class has been renamed or split since this assertion was created.

Test Plan

Compiled locally, nothing more. Relying on CI given the triviality of this change.

I assume that the class has been renamed or split since this
assertion was created.
Copy link
Member

@charliermarsh charliermarsh left a comment

Choose a reason for hiding this comment

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

Thanks!

@charliermarsh charliermarsh marked this pull request as ready for review December 13, 2025 22:36
@astral-sh-bot
Copy link

astral-sh-bot bot commented Dec 13, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@charliermarsh charliermarsh merged commit c7eea1f into astral-sh:main Dec 13, 2025
36 of 37 checks passed
@PeterJCLaw PeterJCLaw deleted the fix-debug-assert-message branch December 13, 2025 23:42
dcreager added a commit that referenced this pull request Dec 15, 2025
* origin/main:
  Update MSRV to 1.90 (#21987)
  [ty] Improve check enforcing that an overloaded function must have an implementation (#21978)
  Update actions/checkout digest to 8e8c483 (#21982)
  [ty] Use `ParamSpec` without the attr for inferable check (#21934)
  [ty] Emit diagnostic when a type variable with a default is followed by one without a default (#21787)
  [ty] Fix callout syntax in configuration mkdocs (#1875) (#21961)
  Update debug_assert which pointed at missing method (#21969)
  [ty] Add support for `__qualname__` and other implicit class attributes (#21966)
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