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

fix(blog): repl code with new duktape args builder #420

Merged
merged 7 commits into from
Aug 16, 2024
Merged

Conversation

RetricSu
Copy link
Collaborator

@RetricSu RetricSu commented Aug 2, 2024

No description provided.

Copy link

vercel bot commented Aug 2, 2024

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

Name Status Preview Comments Updated (UTC)
nervos-ckb-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 14, 2024 1:16pm

@RetricSu RetricSu changed the title fix: update blog-3 with js repl and new duktape fix(blog): repl code with new duktape args builder Aug 14, 2024
@RetricSu RetricSu marked this pull request as ready for review August 14, 2024 13:20
@RetricSu RetricSu changed the base branch from develop to v2 August 16, 2024 01:11
@RetricSu RetricSu merged commit 9f7ffb6 into v2 Aug 16, 2024
4 checks passed
@RetricSu RetricSu deleted the update-blog branch August 16, 2024 01:12
> cd /code
> echo "var a = 1;\nvar b = a + 2;" >> test.js
> ./build/native_args_assembler -f test.js
330000000c0000002f000000000000001b0000007661722061203d20313b5c6e7661722062203d2061202b20323b0a04000000

Choose a reason for hiding this comment

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

js code in docker

var a = 1;\nvar b = a + 2;
  • cause return 101
  • remove \n return success

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