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 20200121 #355

Merged
merged 4 commits into from
Jan 27, 2020
Merged

Integration Candidate 20200121 #355

merged 4 commits into from
Jan 27, 2020

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
1 participant