Skip to content

Fix/228#362

Merged
cre8 merged 6 commits intoopenwallet-foundation:mainfrom
cre8:fix/228
Jan 28, 2026
Merged

Fix/228#362
cre8 merged 6 commits intoopenwallet-foundation:mainfrom
cre8:fix/228

Conversation

@cre8
Copy link
Contributor

@cre8 cre8 commented Jan 27, 2026

closes #228

adds safeverify that will run the checks in sequence without aborting at the first fail.

Signed-off-by: Mirko Mollik mirko.mollik@eudi.sprind.org

cre8 added 2 commits January 27, 2026 08:50
Signed-off-by: Mirko Mollik <mirko.mollik@eudi.sprind.org>
Signed-off-by: Mirko Mollik <mirko.mollik@eudi.sprind.org>
@cre8
Copy link
Contributor Author

cre8 commented Jan 27, 2026

@TimoGlastra may you have a look at this since this was a feature requested by you :) It's not a breaking change since the default approach is not touched

Copy link
Member

@lukasjhan lukasjhan left a comment

Choose a reason for hiding this comment

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

Looks good.

I just like to make sure, let's discuss the interface before v1.0 :)

Signed-off-by: Mirko Mollik <mirko.mollik@eudi.sprind.org>
cre8 and others added 2 commits January 27, 2026 09:49
Signed-off-by: Mirko Mollik <mirko.mollik@eudi.sprind.org>
@cre8 cre8 merged commit c96a9a0 into openwallet-foundation:main Jan 28, 2026
11 checks passed
@cre8 cre8 deleted the fix/228 branch January 28, 2026 07:07
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.

Feat: return list of validations instead of throwing errors

3 participants