-
Notifications
You must be signed in to change notification settings - Fork 101
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
ci: release extism.dll.lib and extism.dll.a #633
Conversation
Here is a sample project that shows this works: |
Alright, it seems to be working: https://github.com/extism/extism/actions/runs/7196310260 Only problem is the |
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.
LGTM, thank you! At some point we should test building with mingw. It's good to have all the import libraries.
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.
LGTM! Great work.
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.
Github didn't clear my notifications with that last LGTM, so you get two LGTMs from me! 😅
Related to #141 and #584
Follow up of #632