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

fix: remove code generation entirely #200

Merged
merged 2 commits into from
Feb 4, 2022
Merged

Conversation

jkelleyrtp
Copy link
Member

@jkelleyrtp jkelleyrtp commented Feb 4, 2022

It's a nice idea to have tsc generate our js binding files, but it leads to weird changes in PRs and noticeably slows down compilation.

This PR moves the binding code into the jsinterpreter crate so we can service both desktop and web from the same crate and same single .js file.

It seems that other projects that use bindings like this just write all their bindings in JS with heavy linting, so that's what we're going to follow until it becomes impossible.

Also, as a side-effect, we've fixed #173

@jkelleyrtp jkelleyrtp requested review from autarch and t1m0t February 4, 2022 06:58
@t1m0t
Copy link
Contributor

t1m0t commented Feb 4, 2022

we should rerun the tests (workflow has been fixed)

@jkelleyrtp jkelleyrtp merged commit 6f10af0 into master Feb 4, 2022
@jkelleyrtp jkelleyrtp deleted the jk/no-more-buildrs branch February 8, 2022 05:29
@Jessidhia
Copy link

Amusingly the v0.2 release notes mention the interpreter is "now (...) written in TypeScript", but that seems to have no longer been the case for over a month 😇

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.

todomvc.rs checkbox didn't work
3 participants