Integration Candidate: 2020-05-27

[astrogeco]

Describe the contribution
Fix #92

and combines

1. Integration Candidate: 2020-05-27 cFE#729
2. Integration Candidate: 2020-05-27 osal#487
3. Integration Candidate: 2020-05-27 PSP#167
4. Integration Candidate: 2020-05-27 ci_lab#50
5. Integration Candidate: 2020-05-27 sample_app#71
6. Integration Candidate: 2020-05-27 to_lab#44
7. Integration Candidate: 2020-05-27 cFS-GroundSystem#101

Testing performed
https://github.com/nasa/cFS/pull/96/checks

Expected behavior changes
See submodule pull requests

PR #94 - lgtm analysis ignores Ui_* files from cFS-GroundSystem

System(s) tested on
Ubuntu:Bionic

Additional context
None

Contributor Info - All information REQUIRED for consideration of pull request
PR #94 - Leor Bleier, NASA-GSFC
See submodules for other contributors

[lgtm-com]

This pull request fixes 7 alerts when merging 64fe4c1 into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard

[lgtm-com]

This pull request fixes 7 alerts when merging 6dd632a into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard

[lgtm-com]

This pull request fixes 7 alerts when merging f3b303a into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard

[lgtm-com]

This pull request fixes 7 alerts when merging 2e60b4c into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard

[lgtm-com]

This pull request fixes 7 alerts when merging 8c83a0f into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard

[lgtm-com]

This pull request fixes 7 alerts when merging 8d1c4db into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard

[lgtm-com]

This pull request fixes 7 alerts when merging 6730497 into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard

[skliper]

The sample_app test hardcodes checking the version string... probably better to just check format (not the actual value).

[lgtm-com]

This pull request fixes 7 alerts when merging 84a95e1 into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard

[lgtm-com]

This pull request fixes 7 alerts when merging 2b180f5 into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard

[lgtm-com]

This pull request fixes 7 alerts when merging 0b3c934 into d80daf5 - view on LGTM.com

fixed alerts:

• 6 for Local variable hides global variable
• 1 for Missing header guard