BenjamenMeyer commented 3 months ago

Thank you for submitting a pull request and becoming a contributor to the Vega Strike Core Engine.

Please answer the following:

Code Changes:

[ ] Have the PR Validation Tests been run? See https://github.com/vegastrike/Vega-Strike-Engine-Source/wiki/Pull-Request-Validation
[ ] This is a documentation change only
[ ] CI system change

Issues:

none

Purpose:

What is this pull request trying to do? port CI system bug fix from 0.9.x to master
What release is this for? master
Is there a project or milestone we should apply this to? master

The Build Process expects to find the output of the tests when doing a release. It is probably wise to run them all the time whether a PR or a release, so let's do so.

BenjamenMeyer commented 3 months ago

831 but for master

stephengtuggy commented 3 months ago

The one Windows failure so far was due to what appears to be a temporary error downloading the source for boost-chrono. (HTTP 502)

vegastrike / Vega-Strike-Engine-Source

Bug Fix: Always run the unit tests #832

831 but for master