-
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
bump inkwell to 0.1.0 #3537
bump inkwell to 0.1.0 #3537
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.
Looks good to me, great work!
Thank you! |
Is there anything needed from my side to make this PR merged? |
Nothing on your side. We have some issues with github actions not triggering here. |
After some days we have no idea on why tests are not being triggered. |
@hamidrezakp we had to revert this PR in #3563 because a bug in inkwell. Once that's resolved we should be good to merge it again |
Oh, that's okay, sorry for taking your time. |
Bumped
inkwell
dependency to0.1.0
Also i would like to know if you have any plans to use
llvm-sys
with latest version or not, i can try to refactor the code.