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

docs: correct wrong import of PS inside JS in FFI.md #468

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Hi-Angel
Copy link

@Hi-Angel Hi-Angel commented Feb 10, 2025

The … from 'Test' import doesn't work unless some additional setup is done. Presumably that's what the hint related to psc compiler is about, but the documentation gives no detail on such configuration as if it would Just Work™. Well, it doesn't work — what does, however, is using relative path. It works well with both spago run and spago bundle-app (barring the fact that gcd has name clash in Prelude).

So replace the path to make sure it works with the default setup people most likely have.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
The `… from 'Test'` import doesn't work unless some additional setup is done. Presumably this is the hint about `psc` compiler, but the documentation gives no detail on such configuration as if it would Just Work™. Well, it doesn't work — what does, however, is using relative path. It works well with both `spago run` and `spago bundle-app` (barring the fact that `gcd` has name clash in Prelude).

So replace the path to make sure it works with the default setup people most likely have.
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.

None yet

1 participant