Skip to content

[django-import-export] Fix prefly tests - #16195

Merged
srittau merged 1 commit into
python:mainfrom
srittau:fix-get-getactions
Aug 12, 2026
Merged

[django-import-export] Fix prefly tests#16195
srittau merged 1 commit into
python:mainfrom
srittau:fix-get-getactions

Conversation

@srittau

@srittau srittau commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

The base classes of ExportActionModelAdmin are incompatible in the implementation when using Python >= 3.12. pyrefly flags this, but we can't do anything about it in the type stubs.

Fixes CI errors on main.

The base classes of `ExportActionModelAdmin` are incompatible in the
implementation when using Python >= 3.12. pyrefly flags this, but we
can't do anything about it in the type stubs.

@AlexWaygood AlexWaygood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Curious how this happened -- was the immediate trigger that upstream django-stubs changed? Or is pyrefly unpinned in our CI?

@srittau

srittau commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

Curious how this happened -- was the immediate trigger that upstream django-stubs changed?

This is what I think. A new version was released two hours ago. That version has different branches for Python 3.12+ and below. I didn't check, but wouldn't be surprised if these branches were new in this version. Also we curiously did # type: ignore[misc] the same line before.

@srittau
srittau merged commit 5167095 into python:main Aug 12, 2026
74 checks passed
@srittau
srittau deleted the fix-get-getactions branch August 12, 2026 12:58
@AlexWaygood

Copy link
Copy Markdown
Member

who knows what misc represents, though 😄

@srittau

srittau commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

who knows what misc represents, though 😄

Yeah, not a fan of that error code, especially when it's not commented.

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