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.

Sign up for GitHub

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

Jump to bottom

Integration Candidate 2020-03-18 #566

Merged
merged 11 commits into from
Mar 27, 2020
Merged

Integration Candidate 2020-03-18 #566

merged 11 commits into from
Mar 27, 2020

Conversation

astrogeco
Copy link
Contributor

@astrogeco astrogeco commented Mar 23, 2020 •
edited by skliper
Loading

Describe the contribution
Integration Candidate 20200318

Fixes: #49, #261, #403, #531, #536, #540, #550, #557, and #561

Documentation scrubs and a makefile change to report branch coverage

Testing performed
Tested in bundle CI - see nasa/cFS#58

Expected behavior changes
See related PRs:
#562
#555
#558

System(s) tested on
CI Ubuntu: Bionic

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman, NASA-GSFC
Gerardo E. Cruz-Ortiz, NASA-GSFC

skliper and others added 9 commits March 13, 2020 15:27
@skliper
Fix #540, Scrub User's Guide and related comments
784fe09 
NOTE - Comments/documentation only, no logic updates
Also fixes #403, resolves all warnings
Also fixes #536, comment typo in CFE_SB_RcvMsg implementation header
@skliper
Fix #261, Clarify table never loaded error
0f60675 
Documentation/comment change only.
Moved get address specific information from error code to API.
@skliper
Fix #267, Document SysLog vs Event use
7e248ba 
Documentation/comment only
@skliper
Fix #499, Clarify Get/SetMsgTime effect on cmd msg
71df47f 
Partial fix (see also #545), comments/documentation only
@skliper
Fix #531, Doc MID use for CFE_TBL_NotifyByMessage
0c00f71 
Documentation/comments only
Recommends unique MID for inter-app communications
@skliper
Fix #577, Scrub OSAL user's guide
8845570 
Documentation changes only
@skliper
Fix #561, Add branch coverage reporting
c6308dc
@astrogeco
Merge pull request #558 from skliper/fix557-osalguide-scrub
fb3ef1c 
Fix #557, OSAL Guide Scrub
@astrogeco
Merge pull request #562 from skliper/fix561-branch-cov-reporting
13213fd 
Fix #561, Add branch coverage reporting
@astrogeco astrogeco changed the title Integration Candidate 20200318 Integration Candidate 2020-03-18 Mar 24, 2020
@astrogeco astrogeco merged commit 62252d1 into master Mar 27, 2020
@astrogeco astrogeco mentioned this pull request Apr 9, 2020
Merged
@skliper skliper linked an issue May 19, 2020 that may be closed by this pull request
API header documentation doesn't match c file documentation #528
Closed
@skliper skliper added this to the 6.8.0 milestone Jun 1, 2020
@skliper skliper linked an issue Sep 24, 2021 that may be closed by this pull request
Document CFE_ES_WriteToSysLog() can be used when CFE_ES_RegisterApp() fails #267
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Bootes
2 participants
@astrogeco @skliper