Skip to content

Fix missing __getattr__ in FileStorage's interface#2199

Merged
davidism merged 1 commit intopallets:2.0.xfrom
4sfaloth:4sfaloth/fix-file-storage-interface
Aug 6, 2021
Merged

Fix missing __getattr__ in FileStorage's interface#2199
davidism merged 1 commit intopallets:2.0.xfrom
4sfaloth:4sfaloth/fix-file-storage-interface

Conversation

@4sfaloth
Copy link
Copy Markdown
Contributor

Fix missing getattr in FileStorage's interface so that mypy no longer wrongly complains that FileStorage has no attribute called read

fixes #2155

  • fixes #

Checklist:

  • Add an entry in CHANGES.rst summarizing the change and linking to the issue.

@4sfaloth
Copy link
Copy Markdown
Contributor Author

@davidism here it is. I've never really worked with .pyi interfaces before but hopefully it's ok 🙏

@davidism
Copy link
Copy Markdown
Member

Looks like your git email is misconfigured, it doesn't match the one that GitHub knows about. Do you want to fix that?

@4sfaloth 4sfaloth force-pushed the 4sfaloth/fix-file-storage-interface branch from 2e0e456 to 3692b44 Compare July 28, 2021 15:31
@4sfaloth
Copy link
Copy Markdown
Contributor Author

Looks like your git email is misconfigured, it doesn't match the one that GitHub knows about. Do you want to fix that?

thanks for the heads up. Fixed

@davidism davidism added the typing label Aug 6, 2021
@davidism davidism added this to the 2.0.2 milestone Aug 6, 2021
@davidism davidism linked an issue Aug 6, 2021 that may be closed by this pull request
@davidism davidism force-pushed the 4sfaloth/fix-file-storage-interface branch from f097c3b to b156d60 Compare August 6, 2021 14:12
@davidism davidism merged commit 2ccdfb4 into pallets:2.0.x Aug 6, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FileStorage perhaps has incorrect typing metadata

2 participants