Skip to content

Bump django-import-export to 4.3.4 - #13414

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

srittau merged 38 commits into
python:mainfrom
Viicos:update-django-import-export

Conversation

@Viicos

@Viicos Viicos commented Jan 19, 2025 •

Copy link
Copy Markdown
Contributor

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

@Viicos Viicos Jan 20, 2025 •

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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
Copy Markdown
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

github-actions Bot commented Feb 2, 2025

Copy link
Copy Markdown
Contributor

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
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