Skip to content

Add stubs for django-import-export#11709

Merged
JelleZijlstra merged 19 commits intopython:mainfrom
Viicos:django-import-export
Dec 28, 2024
Merged

Add stubs for django-import-export#11709
JelleZijlstra merged 19 commits intopython:mainfrom
Viicos:django-import-export

Conversation

@Viicos
Copy link
Contributor

@Viicos Viicos commented Apr 3, 2024 •
edited
Loading

Was discussed in django-import-export/django-import-export#1783

Checklist:

  • formats
  • templatetags
  • __init__.pyi
  • admin.pyi
  • exceptions.pyi
  • fields.pyi
  • forms.pyi
  • instance_loaders.pyi
  • mixins.pyi
  • resources.pyi
  • results.pyi
  • signals.pyi
  • tmp_storages.pyi
  • utils.pyi
  • widgets.pyi

@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@Viicos Viicos force-pushed the django-import-export branch from 882abe4 to 5b4791a Compare April 4, 2024 19:43
@github-actions

This comment has been minimized.

Sign in to view
@Viicos
More work: widgets
d8c47e3 
`render` and `clean` should be redefined on each subclass
as the return value can change. However, this is usually
not an API users will have to deal with, so not worth
the trouble
@github-actions

This comment has been minimized.

Sign in to view
@Viicos Viicos force-pushed the django-import-export branch from 9e3c783 to 0248aa5 Compare April 14, 2024 09:07
@github-actions

This comment has been minimized.

Sign in to view
@Viicos Viicos force-pushed the django-import-export branch from 60a0ee7 to affd870 Compare April 14, 2024 10:09
@github-actions

This comment has been minimized.

Sign in to view
1 similar comment
@github-actions

This comment has been minimized.

Sign in to view
@Viicos Viicos force-pushed the django-import-export branch from cb688bc to 81ff10c Compare April 14, 2024 10:47
@Viicos
Copy link
Contributor Author

Viicos commented Apr 14, 2024

I know there's a lot being added here, so I'd understand if this can't get merged.

I'm also wondering how to fix the issue when Django is being imported? 🤔

@Viicos Viicos marked this pull request as ready for review April 14, 2024 10:49
@Viicos Viicos changed the title Add base stubs for django-import-export Add stubs for django-import-export Apr 14, 2024
@github-actions

This comment has been minimized.

Sign in to view
@srittau
Copy link
Collaborator

srittau commented Apr 14, 2024 •
edited
Loading

The pyright tests are failing, since tablib has no stubs or type annotations yet. So instead of trying to import it, in typeshed we usually just type alias any imported items, like this:

from _typeshed import Incomplete

Dataset: TypeAlias = Incomplete  # tablib.Dataset

The easiest way to work around the stubtest problem for now is to set skip = true in the [tool.stubtest] table of METADATA.toml. Please add a comment why this is a necessary.

@github-actions

This comment has been minimized.

Sign in to view
@Viicos Viicos force-pushed the django-import-export branch from fed5004 to d78b26f Compare April 15, 2024 07:07
@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@Viicos
Copy link
Contributor Author

Viicos commented Apr 15, 2024

Interesting pytype error..

@srittau
Copy link
Collaborator

srittau commented Apr 15, 2024

Interesting pytype error..

Considering that Library.simple_tag doesn't seem to change the signature, it's probably easiest to just leave it off.

We also need to add django-stubs to the stub uploader allowlist.

srittau added a commit to typeshed-internal/stub_uploader that referenced this pull request Apr 15, 2024
@srittau
Add django-stubs to allowlist
bf9e2bf 
Cf. python/typeshed#11709
@srittau srittau mentioned this pull request Apr 15, 2024
Closed
@github-actions

This comment has been minimized.

Sign in to view
@batiste
Copy link

batiste commented Nov 25, 2024

Any news on this one? I suppose I will just suppress mypy for this package then.
Cheers for the good work already done for this!

@github-actions

This comment has been minimized.

Sign in to view
@github-actions

This comment has been minimized.

Sign in to view
@srittau srittau mentioned this pull request Nov 25, 2024
Closed
@github-actions

This comment has been minimized.

Sign in to view
@intgr intgr mentioned this pull request Dec 22, 2024
Merged
@JelleZijlstra
Copy link
Member

JelleZijlstra commented Dec 28, 2024

There's a few remaining CI issues:

  • pytype: stubs/django-import-export/import_export/templatetags/import_export_tags.pyi (3.12): pytype.load_pytd.BadDependencyError: Can't find pyi for 'register', referenced from 'import_export.templatetags.import_export_tags'. This file uses an attribute of a global as a decorator, which is a rare pattern that pytype may not support. Consider removing the decorator and typing the function to whatever type results from the decorator.
  • pyright: /home/runner/work/typeshed/typeshed/stubs/setuptools/setuptools/compat/py310.pyi:9:81 - error: Unnecessary "# pyright: ignore" rule: "reportMissingImports" (reportUnnecessaryTypeIgnoreComment). I understand this is because this PR introduces a dependency on django-stubs, which in turn pulls some other packages into the environment that make it so we don't need that ignore any more. Let's just remove the pyright ignore.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 28, 2024

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

@Viicos
Copy link
Contributor Author

Viicos commented Dec 28, 2024

Once merged, I can tackle the update in a separate PR

@JelleZijlstra JelleZijlstra merged commit eb8af63 into python:main Dec 28, 2024
@Viicos Viicos deleted the django-import-export branch December 28, 2024 15:57
@Viicos Viicos mentioned this pull request Dec 30, 2024
Closed
hoel-bagard pushed a commit to hoel-bagard/typeshed that referenced this pull request Jan 5, 2025
@Viicos @hoel-bagard
Add stubs for django-import-export (python#11709)
ef409c7
@sobolevn sobolevn mentioned this pull request Jan 15, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Viicos @srittau @batiste @JelleZijlstra