Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Integration Candidate: 2020-04-22 #22

Merged
merged 3 commits into from
May 1, 2020
Merged

Conversation

astrogeco
Copy link
Contributor

@astrogeco astrogeco commented Apr 28, 2020

Describe the contribution

Fixes #19

Testing performed
See PRs
Bundle CI - https://github.com/nasa/cFS/pull/74/checks?check_run_id=637311464

Expected behavior changes
PR #20 - Coverage data make lcov includes the sample_lib code

System(s) tested on
See PRs
Bundle CI - Ubuntu:Bionic

Additional context
Part of nasa/cFS#74

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

jphickey and others added 2 commits April 14, 2020 15:28
OSAL now sets these as UT_COVERAGE_COMPILE_FLAGS, UT_COVERAGE_LINK_FLAGS.
Building and linking the UT executable needs a corresponding update.
Fix #19, Update coverage compile/link flag options
@astrogeco astrogeco marked this pull request as ready for review May 1, 2020 21:21
@astrogeco astrogeco merged commit 20f9135 into master May 1, 2020
@skliper skliper added this to the 1.2.0 milestone Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unit tests need to link with UT_COVERAGE_COMPILE_FLAGS/UT_COVERAGE_LINK_FLAGS
3 participants