Add automated unit test for callback dispatcher, and some refactor. #682
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For allowing unit test compile, restructured folder layout of standalone series project.
Imporved the reliability of copying native steam libraries in scenario of directly reference standalone project.
Akarinnnnn
changed the title
…inst. Operating system is auto detected now, by default, will build against developer's OS. Such value can also overrided by command line build by specify SNetOperatingSystem property. CPU architecture preprocessor symbol is also autodetected. Such changes freed Configuration-Platform pair from specifing macros. As a result, we can have debuggable binaries in unit-testing and some other scenario without modify project file.
Mention community NuGet package there because it's largest package registry of .NET. Another reason is consume this by package just need to click install button, no need to copy files themselves. Another change is mentioned target OS property last commit introduced. Since old configuration based OS targetting is outdated.
|
Automation platform macro definition on development machine in standalone project is done. Also updated readme in standalone folder to mention community provided NuGet package, since it's the major starting point of Steamworks.NET of regular .NET project. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Automated unit test extracted from ValueTask experiment #681.
Following refactors has done:
netstandard2.0andnet8.0to allow wider usage(UWP, Framework 4.5) through this is a left-behind of cleanup.For future refactoring on
Steamworks.NET.Standard.csprojstandalone main project file, I'm preparing on platform directly contribute to preprocessor symbol to reduce repeats. They will look like