-
Notifications
You must be signed in to change notification settings - Fork 111
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
feature(lib/runtime): implement ext_offchain_timestamp
and ext_offchain_sleep_until
#1806
Conversation
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.
lgtm
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.
looks good :D please just update HOST_API_TEST_RUNTIME_URL
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.
actually, tests are failing due to this:
lib/runtime/wasmer/imports_test.go:75:54: time.Now().UnixMilli undefined (type time.Time has no field or method UnixMilli)
lib/runtime/wasmer/imports.go:1565:20: undefined: time.UnixMilli
however time.UnixMilli
is defined: https://pkg.go.dev/time#UnixMilli can you update the go.mod
version to go 1.17? (and the README)
@omar391 can you make sure all the checks pass and merge this in? |
🎉 This PR is included in version 0.6.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
…chain_sleep_until` (ChainSafe#1806) * Issue#1508 First commit * Changed interface name + Added testcase * Removed leftover comments * Updated host api runtime + Added testcase * add support for polkadot runtime v0.9.10 * cleanup * Removed testcases temporarily * Updated base Go version to 1.17 to resolve time.UnixMilli issues * Updated go.mod * Removed unused context * Updated go-version to 1.17.x in github workflows * Updated go version in the Dockerfile * added .devcontainer in the gitignore * Linked related issue for the unimplemented unit test * Fixed CGO issues * Fixed lint issues * Fixed unit tests Co-authored-by: noot <elizabethjbinks@gmail.com> Co-authored-by: noot <36753753+noot@users.noreply.github.com>
Changes
ext_offchain_timestamp
methodext_offchain_sleep_until
methodRuntimeStorageAPI
interface intoOffchainStorageAPI
Tests
Issues
ext_offchain_timestamp
but it seems there are some issues with FFI calling which I need to discuss with the teamext_offchain_timestamp_version_1
#1795, implement remaining offchain worker API #1508, Implement unit test forext_offchain_timestamp_version_1
andext_offchain_sleep_until_version_1
#1902Primary Reviewer