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

Fix #138, Convert int32 return codes and variables to CFE_Status_t #139

Conversation

thnkslprpt
Copy link
Contributor

Checklist

Describe the contribution

Testing performed
GitHub CI actions all passing successfully.
Local testing with full cFS suite passing all tests and showing not change to coverage.

Expected behavior changes
No change to behavior.
CFE_Status_t is more expressive and improves consistency with cFE/cFS.

Contributor Info
Avi Weiss @thnkslprpt

@thnkslprpt thnkslprpt force-pushed the fix-138-convert-int32-return-codes-to-CFE_Status_t branch from 75c1136 to 884199a Compare November 1, 2023 18:47
@jphickey jphickey added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Nov 2, 2023
@dzbaker dzbaker added CCB:Approved Indicates code approval by CCB and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Nov 2, 2023
@dmknutsen dmknutsen merged commit 01a4d30 into nasa:main Nov 8, 2023
11 checks passed
@thnkslprpt thnkslprpt deleted the fix-138-convert-int32-return-codes-to-CFE_Status_t branch November 9, 2023 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates code approval by CCB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ci_lab int32 return codes and variables should be converted to CFE_Status_t
4 participants