Skip to content

Bump django-import-export to 4.3.4#13414

Merged
srittau merged 38 commits intopython:mainfrom
Viicos:update-django-import-export
Feb 3, 2025
Merged

Bump django-import-export to 4.3.4#13414
srittau merged 38 commits intopython:mainfrom
Viicos:update-django-import-export

Conversation

@Viicos
Copy link
Contributor

@Viicos Viicos commented Jan 19, 2025

Closes #13328.

This was tedious but pretty straightforward to do. I went over every PR in https://django-import-export.readthedocs.io/en/stable/changelog.html, and updated the stubs accordingly commit per commit.

Viicos added 30 commits January 19, 2025 12:14
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

from typing import IO, Any, ClassVar
from typing_extensions import Self, TypeAlias

Dataset: TypeAlias = Incomplete # tablib.Dataset
Copy link
Contributor Author

@Viicos Viicos Jan 20, 2025

Choose a reason for hiding this comment

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

@srittau, as per #12990 (comment), do you think we can do the same here?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Sure, let's try it!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Seems like it doesn't work with pyright, as we make use of the imported symbol in annotations

@github-actions

This comment has been minimized.

@Viicos Viicos force-pushed the update-django-import-export branch from e498f5e to 1aa4152 Compare January 21, 2025 16:10
@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2025

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

@srittau srittau merged commit e26219d into python:main Feb 3, 2025
43 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