-
-
Notifications
You must be signed in to change notification settings - Fork 262
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
Improve wasi support for gix-config #1347
base: main
Are you sure you want to change the base?
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.
Thanks a lot, this looks good already!
I have a few questions, but don't think there is much in the way of merging once these are answered.
with: | ||
toolchain: nightly | ||
targets: wasm32-wasi | ||
- name: "check" |
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.
Would it be possible to integrate this check with the one that exists?
Probably gix-fs
and gix-sec
should also be tested explicitly then.
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.
gix-fs
and gix-sec
are added with --all-features
flag
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.
Have you seen the other check? Is there any chance this one here can be integrated into the other one?
Have you seen this comment? |
gix-config
is used by https://github.com/nrwl/nx, I'm trying to help nx to compile withwasm32-wasip1-threads
target so that it can be used on the StackBlitz.This pull request is improving the
gix-config
wasi compatibility, and also introduces a CI to check thegix-config
withwasm32-wasi
target.