-
Notifications
You must be signed in to change notification settings - Fork 31
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/del rot smid rmid tests #254
Fix/del rot smid rmid tests #254
Conversation
…e dip process by the delegator Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
…sed credentialing design for IdentifierResource end usage Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
…ignify-ts delegation updates Signed-off-by: 2byrds <[email protected]>
…ts that do rotation Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
…ation idempotent Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #254 +/- ##
=======================================
Coverage 93.06% 93.06%
=======================================
Files 36 36
Lines 7121 7198 +77
=======================================
+ Hits 6627 6699 +72
- Misses 494 499 +5 ☔ View full report in Codecov by Sentry. |
Signed-off-by: 2byrds <[email protected]>
… keripy image Signed-off-by: 2byrds <[email protected]>
There is a small amount of overlap with https://github.com/WebOfTrust/signify-ts/pull/234/files but we'll adjust once either is merged. |
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
Signed-off-by: 2byrds <[email protected]>
@pfeairheller I have made all the updates based on your review. I also added a small change to grouping.py to address @daviddm issue and corresponding PR which contains an additional signify-ts test (WebOfTrust/signify-ts#258) that will now pass and can be merged after this. I also ran against all original singify-ts integration tests again: |
Catches KERIA up with KERIpy changes AND provides all functionality necessary for SIGNIFY-TS integration tests to pass, especially multisig-vlei-issuance but also fixes for rotation, salty, and delegation