Skip to content

Conversation

@mikemoraned
Copy link
Contributor

@mikemoraned mikemoraned commented Jul 12, 2025

  • use add.add, not just add, in Node example
  • fix "AKA" path on reverse-string

(See commits for more details)

in "Running a Component from JavaScript Applications (including the Browser)" section, what is exported from `./dist/transpiled/adder.js` is an `add` object and not a function.

You can see this difference in the example application at https://github.com/bytecodealliance/jco/blob/b2e8641d15dbc25a1d68b66451ab670a47a963d5/examples/components/adder/run-transpiled.js
@mikemoraned mikemoraned changed the title July 2025 small corrections Small corrections to Javascript section Jul 12, 2025
@mikemoraned mikemoraned marked this pull request as ready for review July 12, 2025 20:35
I'm not 100% sure of this, but I think this full `example:string-reverse/reverse.reverse-string` path is meant to break down as:
* package: `example:string-reverse`
* interface: `reverse`
* function name: `reverse-string`

If so, then the first part should be `example:string-reverse` and not `example:reverse-string`
@mikemoraned mikemoraned force-pushed the july2025-small-corrections branch from d120435 to c24b5db Compare July 12, 2025 20:37
Copy link
Collaborator

@vados-cosmonic vados-cosmonic left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Thanks for fixing this @mikemoraned !

@vados-cosmonic vados-cosmonic merged commit dc66801 into bytecodealliance:main Jul 13, 2025
8 checks passed
@mikemoraned mikemoraned deleted the july2025-small-corrections branch July 13, 2025 12:17
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