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

Fix SignaturePacket.verify() Typescript definition #1494

Merged
merged 3 commits into from Feb 18, 2022
Merged

Fix SignaturePacket.verify() Typescript definition #1494

merged 3 commits into from Feb 18, 2022

Conversation

IvanPizhenko
Copy link
Contributor

@IvanPizhenko IvanPizhenko commented Feb 10, 2022

According to doc comment in signature.js:

  /**
   * verifies the signature packet. Note: not all signature types are implemented
   * ...
   * @param {String|Object} data - Data which on the signature applies
   * ...
   */

But it is Uint8Array currently in the openpgp.d.ts.

@IvanPizhenko
Copy link
Contributor Author

IvanPizhenko commented Feb 10, 2022

@larabr It have got strange CI failure, unrelated to the code I've changed, please check https://github.com/openpgpjs/openpgpjs/runs/5148415229?check_suite_focus=true

openpgp.d.ts Outdated Show resolved Hide resolved
IvanPizhenko and others added 2 commits February 18, 2022 13:41
Co-authored-by: larabr <larabr+github@protonmail.com>
@IvanPizhenko
Copy link
Contributor Author

@larabr Updated that doc comment

Copy link
Collaborator

@larabr larabr left a comment

Choose a reason for hiding this comment

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

Thanks!

@larabr larabr merged commit e5c7c77 into openpgpjs:main Feb 18, 2022
@IvanPizhenko IvanPizhenko deleted the fix-signature-packet-verify-ts-def branch February 18, 2022 21:01
@IvanPizhenko
Copy link
Contributor Author

@larabr when do you plan to publish a new release with this thing included?

@twiss
Copy link
Member

twiss commented Mar 2, 2022

@IvanPizhenko I've released v5.2.0 :)

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.

None yet

3 participants