Skip to content

Conversation

@astrogeco
Copy link
Contributor

@astrogeco astrogeco commented Feb 3, 2022

Describe the contribution

PR #2042

PR #2041

PR #2037

PR #2035

Expected behavior changes
See PRs

Testing performed

cFE Checks https://github.com/nasa/cFE/pull/2043/checks
cFS Bundle Checks https://github.com/nasa/cFS/pull/414/checks

Note that CodeQL workflow will fail until nasa/cFS#414 is merged. Passing test can be seen in #2035

System(s) tested on
Ubuntu 18.04

Additional context
Part of nasa/cFS#414

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
@skliper

Jacob Hageman and others added 8 commits January 25, 2022 16:04
If no return value override is set test against CFE_SB_INVALID_MSG_ID
Fix #2036, Revert breaking change to CFE_ES_AppInfo_t structure
- Add new parameters to match updated cFS-CodeQL workflow interface
introduced in nasa/cFS#413
- Add file-exclusion checks to Action trigger so workflow doesn't run if
only changes in commit or pull request are to documentation
…vement

Fix #2040, Improve CFE_SB_IsValidMsgId handler
…macro

Fix #2039, Replace CFE_SB_ValueToMsgId(0) with CFE_SB_INVALID_MSG_ID
@astrogeco astrogeco marked this pull request as ready for review February 3, 2022 21:03
@astrogeco astrogeco merged commit 99429bf into main Feb 3, 2022
@skliper skliper added this to the Draco milestone Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment