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.
os:mono
andos:windows
are behaving wrongly for .Net Core version of the Paket. This is caused by the fact the check depends onisMonoRuntime
function which on .Net Core always returnfalse
which causedos:mono
to never be matched, andos:windows
be matched on all platforms.The workaround for the problem is to use
os:win
,os:linux
andos:osx
- but it's breaking change for anyone already using those checks who just wanted to modernize build infrastructure (👋).Fix makes old syntax works correctly also on .Net Core by using the same checks as for
os:win
,os:osx
andos:linux
.os:windows
will behave just likeos:win
andos:mono
will beos:osx || os:linux