search

nasa / bplib

Apache License 2.0
27 stars 13 forks source link

Fix #226, make workflow error in ctest visible #227

Closed jphickey closed 1 year ago

jphickey commented 1 year ago

Describe the contribution Use "tee" to display the ctest output which should in turn be visible in the workflow run log. If this fails, this should provide the details of what failed.

Fixes #226

Testing performed Run workflow, check output. Force an failure to confirm expected output.

Expected behavior changes Workflow should show output of ctest.

System(s) tested on Github hosted workflow

Contributor Info - All information REQUIRED for consideration of pull request Joseph Hickey, Vantage Systems, Inc.

jphickey commented 1 year ago

Original PR (commit dc2ec2c) had an intentional forced error. Workflow run can be seen here:

https://github.com/nasa/bplib/actions/runs/3696394219/jobs/6260044207

Forced error is removed in 7abd9fa.

karl-nasa commented 1 year ago

Approved.

jphickey commented 1 year ago

Reviewed at 2022-12-14 CCB and approved for merge