Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Update submodules #89
Update submodules #89
Changes from 25 commits
b4a6ce6
3dcf78b
ab91d28
565f33c
ad641c6
69d34b7
ecfb490
85e8706
1d72604
6af507a
8b2452f
e5ee196
8b7f5ff
270e7df
7d806d9
46bd983
1a58640
7dfa2d6
c9827ac
fefca3f
ab2493a
6185d44
91a5197
21a0895
f00dfc0
c7040c3
da42027
5dbcc56
1e821f0
02d54b8
bd2022e
41b9b25
759548c
ca78391
d817d11
9d900fb
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
not your fault, but
ilammy/msvc-dev-cmd
<-- in my experience, this github action has no effectit ended up in all our ci.yml files when we were first setting up our repos and copy/pasting everything
later we discovered it had no effect, but we've never made a concerted effort to fix this across our 20+ repos :(
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.
https://tenor.com/en-GB/view/coding-programming-pink-panther-when-you-delete-a-block-of-code-that-you-though-was-useless-gif-17338075
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.
not your fault, but this "windows-vc14" run isn't doing what it claims.
vc14 means version 14.0 of the compiler, aka toolset 140, which came with Visual Studio 2015, but you can still use those older toolset versions with later versions of Visual Studio
but it's actually just using the default compiler on this machine, which is Visual Studio 2019's version 14.29
Here's the line in the logs where you see what it's actually using.
and then you cross-reference that against the Wikipedia page that details what the hell those version numbers mean. And you see that 14.29 means it's the latest build Visual Studio 2019 compiler
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.
Added a TODO for now.