-
-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
Test minimal windows build using Github actions #14737
Conversation
b545b5f
to
e6ad662
Compare
aded8d9
to
1e73259
Compare
Might í suggest splitting up the tasks in several steps instead of one single run? |
I was doing this because the ilammy/msvc-dev-cmd@v1 does the MSDEV command line prompt.. I wasnt sure how persistent things were.. |
1d70e05
to
fc2284c
Compare
Woohoo - minimal windows build and test in 13 minutes.. |
fc2284c
to
e3f9a84
Compare
e3f9a84
to
d9d1a7f
Compare
This is quite readable now. |
24 hours has passed since 'approval: done' was set, but this PR has failing CI tests. Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually. |
This pull request is ready to merge |
Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from #14737)
Thanks for the improvements @levitte. Merged to master. |
Checklist