Skip to content

correct type bounds for base signing provider#282

Draft
iaik-jheher wants to merge 3 commits intodevelopmentfrom
fix/signer-bounds
Draft

correct type bounds for base signing provider#282
iaik-jheher wants to merge 3 commits intodevelopmentfrom
fix/signer-bounds

Conversation

@iaik-jheher
Copy link
Collaborator

PlatformSigningProvider should require PlatformSigningKeyConfigurationBase, but SigningProvider should only require SigningProviderSigningKeyConfigurationBase.

Copy link
Collaborator

@JesusMcCloud JesusMcCloud left a comment

Choose a reason for hiding this comment

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

seems legit

@iaik-jheher iaik-jheher marked this pull request as draft May 12, 2025 17:02
@JesusMcCloud JesusMcCloud force-pushed the development branch 2 times, most recently from ee39ea6 to 62049e4 Compare June 2, 2025 09:56
Copy link
Collaborator

@JesusMcCloud JesusMcCloud left a comment

Choose a reason for hiding this comment

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

Well, if the tests can be made to work again, I'll re-review

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