Skip to content

Conversation

@b1nhack
Copy link

@b1nhack b1nhack commented May 17, 2024

This Pull Request fixes/closes #2188, #2198.

It changes the following:

  • Adds support for external bin signing with ssh keys

I followed the checklist:

  • I added unittests
  • I ran make check

thread 'tests::test_pre_commit_py' panicked at git2-hooks/src/lib.rs:471:9:
assertion failed: res.is_ok()

  • I tested the overall application
  • I added an appropriate item to the changelog

@extrawurst
Copy link
Collaborator

since this basically undos the way ssh signing worked since 945ec45 i assume we can cleanup some of the now not used dependencies?

@b1nhack
Copy link
Author

b1nhack commented May 20, 2024

Okay, I'll check the code and try to make more optimizations.

@b1nhack
Copy link
Author

b1nhack commented May 22, 2024

No unused dependencies, but did some other code cleanup, fixed a test.

@b1nhack b1nhack closed this by deleting the head repository Jun 9, 2024
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.

[ssh signing] support custom signing program

3 participants