Skip to content

Integration Candidate 20200121#355

Merged
skliper merged 4 commits intomasterfrom
ic-20200121
Jan 27, 2020
Merged

Integration Candidate 20200121#355
skliper merged 4 commits intomasterfrom
ic-20200121

Conversation

@skliper
Copy link
Contributor

@skliper skliper commented Jan 21, 2020

Describe the contribution
Fix #328, fix #329

Testing performed
Steps taken to test the contribution:

  1. Current CI passes
  2. Enhanced CI in OSAL should use UT framework similar to that of CFE #40 passes debug/with deprecated elements testing

Expected behavior changes
Resolved LGTM errors, no functional changes

System(s) tested on

  • CI: Ubuntu 18.04

Additional context
N/A

Third party code
N/A

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added this to the 5.1.0 milestone Jan 21, 2020
@skliper
Copy link
Contributor Author

skliper commented Jan 27, 2020

CCB 20200122 - Reviewed and approved for merge to master

@skliper skliper added the CCB:Approved Indicates code review and approval by community CCB label Jan 27, 2020
@skliper skliper merged commit 89de6b0 into master Jan 27, 2020
@skliper skliper deleted the ic-20200121 branch January 27, 2020 15:00
jphickey added a commit to jphickey/osal that referenced this pull request Aug 10, 2022
Adds a "global_build_options.cmake" file akin to the existing
arch_build/mission_build option files.  Include an example of
this file that optionally does add_definitions() to omit the
deprected elements for build testing.
jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
Fix nasa#355, Add global scope option to omit deprecated items
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CCB:Approved Indicates code review and approval by community CCB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

lgtm warning - misc. osal lgtm warning

2 participants