Skip to content
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

Update multisig-vault for clarinet 2 #42

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kilianmh
Copy link

Project was generated with following commands:
clarinet new multisig-vault
clarinet contract new multisig-vault

multisig-vault.clar copied unchanged.
.gitignore file removed
The tests have been moved to multisig-vault.test.ts and changed to work with clarinet-sdk and vitest.

For the tests to run the user has to first execute
npm install
before the tests can be performed with
npm test

Probably some adjustments have to be made still. Just tell me and I'll try to fix it soon so we have a working example since clarinet test is not working anymore in Clarinet 2.0
See #41

Update the multisig-vault project for clarinet 2.
Project was generated with following commands:
clarinet new multisig-vault
clarinet contract new multisig-vault

.gitignore file removed
multisig-vault.clar is unchanged.
The tests have been moved to multisig-vault.test.ts and
changed to work with clarinet-sdk and vitest.
Copy link

netlify bot commented Nov 14, 2023

Deploy Preview for claritybook ready!

Name Link
🔨 Latest commit 71e61e3
🔍 Latest deploy log https://app.netlify.com/sites/claritybook/deploys/65536b4d46681600080eee33
😎 Deploy Preview https://deploy-preview-42--claritybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@MarvinJanssen
Copy link
Member

Amazing, thank you for your contribution! Do you mind updating the other tests as well? It definitely needs to be fixed but merging only makes sense if it fixes all sections about testing.

Don't forget to add yourself to the contributors listed on the title page (if you want).

https://github.com/clarity-lang/book/blob/main/src/title-page.md

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