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 the build and other improvements #38

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

tritao
Copy link

@tritao tritao commented Oct 2, 2023

Hey,

First of all thanks for all the work on this. Just recently got exploring the code and what's there seems really cool.

But realized that some stuff seems broken out-of-the-box, which I've tried to fix it and make it work best to my knowledge.

CI also seems broken for the last few months, which makes me question, are you planning to keep maintaining this package? Need any help?

@juancastillo0
Copy link
Owner

Hi, thank you for your work!

The latest work is in this branch #14 with multiple changes primarily related to wit and components, but there are other changes. For example, I was also working on the flutter_rust_bridge update there following #36. I think it will be better to merge that PR instead, since I had made other changes which may be related. However, I will evaluate your changes and maybe merge it after that one. Thanks again and sorry for the delay. Also the flutter_rust_bridge errors seem to be a breaking change on their part, I think I will release a version with a version restriction for that package.

CI has been kind of challenging to setup, specially android emulators and macos which errors out at a predefined Flutter timeout for integration tests (flutter/flutter#105913). Other errors are related to experimental features such as multitheading in some platforms, but there are plenty of tests that should work.

Yes, I plan on maintaining the package. However, wit and wasm components have taken most of my time. Any help is, of course, appreciated!

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.

2 participants