-
Notifications
You must be signed in to change notification settings - Fork 824
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
chore(runtime-core) Some code clean ups #1226
chore(runtime-core) Some code clean ups #1226
Conversation
Those are not necessary for the user. It's only internal implementation, that must be public because it ends up in public types, but the compiler infers everything automatically.
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 to me!
bors r+ |
This PR was included in a batch with a merge conflict, it will be automatically retried |
Canceled |
bors r+ |
This PR was included in a batch with a merge conflict, it will be automatically retried |
1 similar comment
This PR was included in a batch with a merge conflict, it will be automatically retried |
Merge conflict |
bors r+ |
1226: chore(runtime-core) Some code clean ups r=syrusakbary a=Hywan Extracted from #1018. Opinionated patch. Some code clean ups, mostly renaming `ExternalFunction` to `HostFunction`. Co-authored-by: Ivan Enderlin <[email protected]> Co-authored-by: Ivan Enderlin <[email protected]> Co-authored-by: Syrus Akbary <[email protected]>
Build succeeded
|
Extracted from #1018. Opinionated patch.
Some code clean ups, mostly renaming
ExternalFunction
toHostFunction
.