Fix #78, Convert int32 return codes and variables to CFE_Status_t

[thnkslprpt]

Checklist

• I reviewed the Contributing Guide.
• I signed and emailed the appropriate Contributor License Agreement to GSFC-SoftwareRelease@mail.nasa.gov and copied cfs-program@lists.nasa.gov.

Describe the contribution

• Fixes CS int32 return codes and variables should be converted to CFE_Status_t #78
  • int32 return codes converted over to CFE_Status_t (incl. in the test code)
  • int32 status/return variables holding cFE return codes converted to CFE_Status_t (incl. in the test code)
  • removed the CS_SUCCESS macro (and replaced with CFE_SUCCESS) which was in the "error codes" section (there were only a few uses anyway). It seems clearer to just use CFE_SUCCESS for all success returns in the app and from calls to CFE etc.
  • removed some trailing whitespace from the yaml files to avoid any future warnings

Does CS_SUCCESS need to be deprecated even though it's equivelent to CFE_SUCCESS, i.e. == 0)?

Testing performed
GitHub CI actions all passing successfully.

Expected behavior changes
No change to behavior (no types have actually changed with this PR).
CFE_Status_t is more expressive and improves consistency with cFE/cFS.

Contributor Info
Avi Weiss @thnkslprpt

[github-advanced-security]

CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

[chillfig]

Looks Good!