-
Notifications
You must be signed in to change notification settings - Fork 5
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
ci: add github actions for testing #4
Conversation
Signed-off-by: Adam Reese <adam@reese.io>
- name: Setup Wasmtime | ||
uses: bytecodealliance/actions/wasmtime/setup@v1 | ||
with: | ||
version: "13.0.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm, do we want to use one of the latest version of wasmtime maybe?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need wasmtime 13 because it's the latest version that works with TinyGo 0.28. TinyGo 0.29 is when net/http
was removed and all the Transport
bits in the client stopped working. I'll a some note about somewhere
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Specifically --map-dir
was removed from wasmtime in v14
- name: Setup TinyGo | ||
uses: acifani/setup-tinygo@v2 | ||
with: | ||
tinygo-version: '0.28.0' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about TinyGo 0.33.0 here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See above comment #4 (comment)
- name: Setup Go | ||
uses: actions/setup-go@v5 | ||
with: | ||
go-version: '1.20' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps Go 1.23 here?
No description provided.