Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

sample_app Integration Candidate: 2020-11-24 #113

Merged
merged 2 commits into from
Dec 2, 2020
Merged

Conversation

astrogeco
Copy link
Contributor

@astrogeco astrogeco commented Dec 1, 2020

Describe the contribution
Fix #106, Refactor UT_SetForceFail to UT_SetDefaultReturnValue

Testing performed
See events for "github actions test" at https://github.com/astrogeco/cFS/actions

Expected behavior changes

PR #107- Rename UT_SetForceFail to UT_SetDefaultReturnValue since some functions that retain more than 1 value are not necessarily failing

System(s) tested on
Ubuntu 18.04

Additional context
Part of nasa/cFS#158

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
@zanzaben

@astrogeco astrogeco force-pushed the integration-candidate branch from 5fe5693 to 093e838 Compare December 1, 2020 23:18
New function name is UT_SetDefaultReturnValue
@astrogeco astrogeco force-pushed the integration-candidate branch from 093e838 to 473e7d4 Compare December 1, 2020 23:25
@astrogeco astrogeco marked this pull request as ready for review December 2, 2020 14:32
Integration Candidate 2020-11-24
@astrogeco astrogeco merged commit c222367 into main Dec 2, 2020
astrogeco added a commit to nasa/cFS that referenced this pull request Dec 3, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor UT_SetForceFail
2 participants