-
Notifications
You must be signed in to change notification settings - Fork 71
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
chore: maintenance update js wrapper #296
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Timo Glastra <[email protected]>
Signed-off-by: Timo Glastra <[email protected]>
Signed-off-by: Timo Glastra <[email protected]>
Signed-off-by: Timo Glastra <[email protected]>
Signed-off-by: Timo Glastra <[email protected]>
wrappers/javascript/indy-vdr-nodejs/tests/AcceptanceMechanismsRequest.test.ts
Outdated
Show resolved
Hide resolved
Also if it's not too much trouble do you mind incorporating the changes in from this PR? #270 Just the changes from these files:
otherwise I can do it in a separate PR if you'd prefer |
Yes, will do! |
Signed-off-by: Timo Glastra <[email protected]>
@berendsliedrecht @andrewwhitehead any idea why the macOS build is suddenly failing? Only made changes to the JS wrapper |
@TimoGlastra not sure why it's just failing for the mac build, try updating |
Signed-off-by: Berend Sliedrecht <[email protected]>
Signed-off-by: Timo Glastra <[email protected]>
Still failing sadly @berendsliedrecht |
Now sure what is happening with the segmentation vault now ... :/ |
@TimoGlastra could it be related to vitest? |
Yeah maybe 🤷 Haven't found the time to look into it yet :( |
Signed-off-by: Timo Glastra <[email protected]>
Signed-off-by: Timo Glastra <[email protected]>
Updates the JS wrapper: