Skip to content

Feat/clarinet sdk#51

Closed
hugoclrd wants to merge 3 commits into
clarity-lang:mainfrom
hugoclrd:feat/clarinet-sdk
Closed

Feat/clarinet sdk#51
hugoclrd wants to merge 3 commits into
clarity-lang:mainfrom
hugoclrd:feat/clarinet-sdk

Conversation

@hugoclrd

@hugoclrd hugoclrd commented May 3, 2024

Copy link
Copy Markdown
Contributor

Description

This is a proposal to use the clarinet-sdk (beta version of @hirosystems/clarinet-sdk-browser) instead of the outdated repl that's in place.

It's not a 1 to 1 replacement.
The previous version had some cool feature the simnet doesn't have at the moment. But improvements can be expected in the simnet for this kind of usage (docs and repl in the browser).

What this PR does

  • Remove the embedded repl
  • Add @hirosystems/clarinet-sdk-browser
  • Use await simnet.initEmtpySession(); and simnet.runSnippet(...) to execute code

Misc

  • Modernize the repo and its dependencies
  • Started to add some JSDoc types

Sorry this PR became bigger than just using the simnet.

@vercel

vercel Bot commented May 3, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
clarity-book ❌ Failed (Inspect) May 3, 2024 6:24pm

@hugoclrd

hugoclrd commented May 4, 2024

Copy link
Copy Markdown
Contributor Author

@MarvinJanssen I can see the my request to access the vercel project was approved, but I can't see the build (I can't see much actually).
Can you tell me what's the error, or see if you can give me access?

@friedger friedger left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@friedger friedger mentioned this pull request May 27, 2024
@friedger

Copy link
Copy Markdown
Collaborator

Replaced by #58

@friedger friedger closed this Jan 19, 2025
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