Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update fpdf2 stubs to 2.6.1 #9546

Merged
merged 11 commits into from
Jan 17, 2023
Merged

Update fpdf2 stubs to 2.6.1 #9546

merged 11 commits into from
Jan 17, 2023

Conversation

srittau
Copy link
Collaborator

@srittau srittau commented Jan 16, 2023

Closes #9542

Comment on lines 152 to 160
# The following definition crashes stubtest.
# def set_encryption(
# self,
# owner_password: str,
# user_password: str | None = None,
# encryption_method: EncryptionMethod | str = ...,
# permissions: AccessPermission = ...,
# encrypt_metadata: bool = False,
# ) -> None: ...
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This needs further investigation for which I didn't have time yet.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I just ran this against stubtest HEAD and this seems to fix it. I'll update the comment.

@github-actions

This comment has been minimized.

@AlexWaygood
Copy link
Member

I think we'll need a pytype release that includes google/pytype@829c589 if you want to keep the string default values — or you could take them out for now, and add them back when we have the new pytype release

@github-actions

This comment has been minimized.

@srittau
Copy link
Collaborator Author

srittau commented Jan 16, 2023

@rchen152 Is there a timeline for the next pytype release? This is non-critical, since we can easily work around the issue, but if a release is due soon, I would rather wait.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

Looks good, just a few points

stubs/fpdf2/fpdf/encryption.pyi Show resolved Hide resolved
stubs/fpdf2/fpdf/encryption.pyi Outdated Show resolved Hide resolved
stubs/fpdf2/fpdf/encryption.pyi Show resolved Hide resolved
stubs/fpdf2/fpdf/line_break.pyi Show resolved Hide resolved
stubs/fpdf2/fpdf/linearization.pyi Outdated Show resolved Hide resolved
stubs/fpdf2/fpdf/html.pyi Show resolved Hide resolved
@github-actions
Copy link
Contributor

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

@srittau srittau merged commit 7b848b4 into python:main Jan 17, 2023
@srittau srittau deleted the fpdf2-2.6.1 branch January 17, 2023 12:25
@rchen152
Copy link
Collaborator

@rchen152 Is there a timeline for the next pytype release? This is non-critical, since we can easily work around the issue, but if a release is due soon, I would rather wait.

Sorry for the delay! (Yesterday was MLK day here.) We usually aim to do one pytype release a week. It's a manual process, so there's no fixed schedule. Yesterday's nightly tests were clean, so I can cut a release right now, actually.

@AlexWaygood
Copy link
Member

Yesterday's nightly tests were clean, so I can cut a release right now, actually.

That would be great, if possible! It would help unblock #9501

@rchen152
Copy link
Collaborator

Done! The new pytype version is 2023.01.17.

@AlexWaygood
Copy link
Member

Done! The new pytype version is 2023.01.17.

Amazing, thank you!

srittau added a commit to srittau/typeshed that referenced this pull request Jan 18, 2023
AlexWaygood pushed a commit that referenced this pull request Jan 18, 2023
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.

3 participants