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

update cFE source and unit tests to use CFE_Status_t #921

Open
CDKnightNASA opened this issue Sep 30, 2020 · 1 comment · May be fixed by #2264
Open

update cFE source and unit tests to use CFE_Status_t #921

CDKnightNASA opened this issue Sep 30, 2020 · 1 comment · May be fixed by #2264
Assignees
@jphickey
Labels
CFS-40 CFS-41 CFS-42 enhancement

Comments

@CDKnightNASA
Copy link
Contributor

Is your feature request related to a problem? Please describe.
As a followon to #888, we should update all of the cFE source to use the CFE_Status_t return status type instead of int32.

Describe alternatives you've considered
May want to do this in phases, per component; or incorporate into a clang-format code cleanup change set.

Requester Info
Christopher.D.Knight@nasa.gov
This was referenced Apr 8, 2021
Closed
Closed
@skliper skliper added the CFS-40 label Apr 13, 2021
@astrogeco astrogeco mentioned this issue Apr 19, 2021
Closed
@skliper skliper added the CFS-41 label Apr 20, 2021
@astrogeco astrogeco mentioned this issue Apr 23, 2021
Closed
@skliper skliper added the CFS-42 label Apr 29, 2021
This was referenced May 14, 2021
Open
Merged
@jphickey
Copy link
Contributor

I ended up basically implementing this as part of doing other reviews, so I will submit the PR, but it should go into the Draco set, not for Caelum.

@jphickey jphickey self-assigned this Jul 23, 2021
@jphickey jphickey mentioned this issue Aug 3, 2021
Closed
@skliper skliper mentioned this issue May 27, 2022
Closed
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Mar 25, 2023
@thnkslprpt
Fix nasa#921, Update remaining cFE source/tests to use CFE_Status_t
da32d56
@thnkslprpt thnkslprpt linked a pull request Mar 25, 2023 that will close this issue
Open
2 tasks
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Mar 31, 2023
@thnkslprpt
Fix nasa#921, Update remaining cFE source/tests to use CFE_Status_t
53fc364
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Jan 25, 2024
@thnkslprpt
Fix nasa#921, Update remaining cFE source/tests to use CFE_Status_t
6d7bca9
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Mar 24, 2024
@thnkslprpt
Fix nasa#921, Update remaining cFE source/tests to use CFE_Status_t
af9e96d
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Apr 20, 2024
@thnkslprpt
Fix nasa#921, Update remaining cFE source/tests to use CFE_Status_t
e32f30c
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Apr 20, 2024
@thnkslprpt
Fix nasa#921, Update remaining cFE source/tests to use CFE_Status_t
74c5e58
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Apr 20, 2024
@thnkslprpt
Fix nasa#921, Update remaining cFE source/tests to use CFE_Status_t
33807fe
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Apr 21, 2024
@thnkslprpt
Fix nasa#921, Update remaining cFE source/tests to use CFE_Status_t
a53a43a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
CFS-40 CFS-41 CFS-42 enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #921, Update remaining cFE source/tests to use CFE_Status_t thnkslprpt/cFE
3 participants
@jphickey @CDKnightNASA @skliper