Skip to content

Made zip type generic#704

Merged
edi9999 merged 1 commit intoopen-xml-templating:masterfrom
oleksandr-danylchenko:specific-zip-type
Mar 3, 2023
Merged

Made zip type generic#704
edi9999 merged 1 commit intoopen-xml-templating:masterfrom
oleksandr-danylchenko:specific-zip-type

Conversation

@oleksandr-danylchenko
Copy link
Copy Markdown
Contributor

@oleksandr-danylchenko oleksandr-danylchenko commented Mar 3, 2023

Previously, all the types of the zip were any. It doesn't allow using of the PizZip or other archivers with type confidence. And IDE pulls types from random places when you try using getZip:
image

In this PR I made the type of passed zip generic and inferrable which will improve the DX and will prevent as PizZip statements in the code.

@edi9999 edi9999 merged commit 2d7792d into open-xml-templating:master Mar 3, 2023
@edi9999
Copy link
Copy Markdown
Member

edi9999 commented Mar 3, 2023

Thanks a lot , very cool merge request, it will indeed be useful to autocomplete after the "getZip()" this way !

Thanks ! Will publish this soon !

@edi9999
Copy link
Copy Markdown
Member

edi9999 commented Mar 5, 2023

Published in 3.34.3

@oleksandr-danylchenko
Copy link
Copy Markdown
Contributor Author

Thank you 🤝

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