Skip to content

Conversation

@chigby
Copy link
Contributor

@chigby chigby commented Nov 17, 2025

Forgot to import this when adding it in my previous PR.

Forgot to import this when adding it in my previous PR.
@chigby chigby force-pushed the fix-unimported-name branch from 627a8b9 to c3be425 Compare November 17, 2025 21:06
Running the pre-commit script generated these changes.
The error:

```
Error: Argument of type "dict[str, int]" cannot be assigned to parameter "value" of type "str" in function "__setitem__"
  "dict[str, int]" is not assignable to "str" (reportArgumentType)
```
@chigby chigby force-pushed the fix-unimported-name branch from 4ce76c7 to bfd1168 Compare November 17, 2025 21:49
@chigby
Copy link
Contributor Author

chigby commented Nov 17, 2025

All checks passing! I had to make a few other slight modifications (both non-functional changes) to convince pre-commit and pyright to give me the "ok!" but that seems like it's happening. @mgax this should be ready to merge if you approve.

Copy link
Member

@mgax mgax left a comment

Choose a reason for hiding this comment

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

Oh yes, dict typing can be fun 🫠

Thank you for taking the time to make the CI happy! 🦸

@mgax mgax merged commit 0edfb9e into wagtail:main Nov 18, 2025
7 checks passed
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