Skip to content

Use signify package from tkey-sign-cli#20

Draft
mchack-work wants to merge 11 commits intomainfrom
signify-methods
Draft

Use signify package from tkey-sign-cli#20
mchack-work wants to merge 11 commits intomainfrom
signify-methods

Conversation

@mchack-work
Copy link
Copy Markdown
Member

Description

Instead of duplicating signing code from tkey-sign-cli we use the package with the necessary types and methods.

See PR tillitis/tkey-sign-cli#38

Type of change

  • Feature (non breaking change which adds functionality)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • I have squashed my work to relevant commits and rebased on main for linear history
  • I have added a "Co-authored-by: x" if several people contributed, either pair programming or by squashing commits from different authors.
  • I have updated the documentation where relevant (readme, dev.tillitis.se etc.)
  • QEMU is updated to reflect changes

agren and others added 10 commits January 23, 2026 11:10
- Simplify type assertion. Since we're already copying the data, do
  the assertion there.

- Add linefeed when reporting error.

- Whitespace fix.

- Use symbol for pubkey size.

- Use sillyCaps pubKey.
Use exported types and functions from tkey-sign-cli to handle
signify-format public keys and signature files.
@mchack-work mchack-work requested a review from agren February 3, 2026 12:18
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