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 #1386, Remove CFE_EVS_Unregister API #1577

Merged

Conversation

skliper
Copy link
Contributor

@skliper skliper commented May 25, 2021

Describe the contribution
Fix #1386, Remove CFE_EVS_Unregister API

  • Removed requirement cEVS3101
  • Removed documentation references
  • Removed from API header
  • Removed stub
  • Removed implementation
  • Removed coverage test

Testing performed
Built and ran unit tests, passed

Expected behavior changes
Removed API.

System(s) tested on

  • Hardware: Intel I5/Docker
  • OS: Ubuntu 18.04
  • Versions: Bundle main + this commit

Additional context
None

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

- Removed requirement cEVS3101
- Removed documentation references
- Removed from API header
- Removed stub
- Removed implementation
- Removed coverage test
@skliper skliper added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label May 25, 2021
@skliper skliper added this to the 7.0.0 milestone May 25, 2021
@skliper skliper added the CCB:Approved Indicates code review and approval by community CCB label May 26, 2021
@astrogeco astrogeco removed the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label May 27, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate June 2, 2021 14:25
@astrogeco astrogeco merged commit 4a3129c into nasa:integration-candidate Jun 2, 2021
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
I also added some NULL checks in CFE_MSG_Set*** functions for the second parameters which could let to comparisson between pointer and unsigned int or other data types or "invalid operands to binary" errors!
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
I also added some NULL checks in CFE_MSG_Set*** functions for the second parameters which could let to comparisson between pointer and unsigned int or other data types or "invalid operands to binary" errors!
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
I also added some NULL checks in CFE_MSG_Set*** functions for the second parameters which could let to comparisson between pointer and unsigned int or other data types or "invalid operands to binary" errors!
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
I also added some NULL checks in CFE_MSG_Set*** functions for the second parameters which could let to comparisson between pointer and unsigned int or other data types or "invalid operands to binary" errors!
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
paulober pushed a commit to paulober/cFE that referenced this pull request Jul 30, 2021
@skliper skliper deleted the fix1386-remove_evs_unregister branch October 22, 2021 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates code review and approval by community CCB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove CFE_EVS_Unregister API
3 participants