-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat(wkg-core): Adds new wkg-core crate with lockfile support #91
feat(wkg-core): Adds new wkg-core crate with lockfile support #91
Conversation
This includes support for an optional config and tests for locking Signed-off-by: Taylor Thomas <taylor@cosmonic.com>
f8bce32
to
c372d0a
Compare
Heads up that the test failure is happening because macos runners don't have docker installed by default. I'll figure that out shortly |
c372d0a
to
20b3895
Compare
This adds a new `wit` subcommand to `wkg` that supports building wit packages and fetching/populating a deps directory. I'm sure there is much more we can do here and some obtuse edge cases that aren't supported, but I did test fetching dependencies for various worlds that used wasi:http and wasi:cli. In a follow up PR, I'll add some more integration tests Signed-off-by: Taylor Thomas <taylor@cosmonic.com>
20b3895
to
f6d712c
Compare
Heads up that I added a commit with integration tests. Update isn't tested yet because we need to be pointing at wasi.dev so we can try an update from 0.2.1, but also that doesn't do much yet (it is essentially the same as fetch but forces a new lock file (for example if you changed registries) |
5d6100f
to
a7be547
Compare
5a477aa
to
afad3f0
Compare
Signed-off-by: Taylor Thomas <taylor@cosmonic.com>
afad3f0
to
9db8f08
Compare
This adds a new
wit
subcommand towkg
that supports building wit packages and fetching/populating a deps directory. I'm sure there is much more we can do here and some obtuse edge cases that aren't supported, butI did test fetching dependencies for various worlds that used wasi:http and wasi:cli so the basics are functional! In a follow up PR, I'll add some more integration tests