Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix #52, Add version information to NOOP event #131

Conversation

thnkslprpt
Copy link
Contributor

@thnkslprpt thnkslprpt commented Oct 22, 2022

Checklist

Describe the contribution

Testing performed
Run/Build and confirmed NOOP command event reporting as expected.
Screenshot 2022-10-22 21 00 58

Expected behavior changes
Adds version information to NOOP command to align with the other cFS components/apps.

System(s) tested on
Intel(R) Celeron(R) N4100 CPU @ 1.10GHz x86_64
Debian GNU/Linux 11 (bullseye)

Contributor Info
Avi Weiss @thnkslprpt

@dzbaker
Copy link
Collaborator

dzbaker commented Oct 27, 2022

CCB 27 October 2022: @dzbaker will review and make sure that formatting is in line with other apps.

@dzbaker dzbaker added this to the Fornax milestone Nov 21, 2022
@dzbaker dzbaker modified the milestones: Fornax, Equuleus Dec 7, 2022
@thnkslprpt thnkslprpt closed this Nov 1, 2023
@thnkslprpt thnkslprpt force-pushed the fix-52-add-version-information-to-noop-event branch from 5c88107 to 2930b94 Compare November 1, 2023 18:52
@thnkslprpt thnkslprpt deleted the fix-52-add-version-information-to-noop-event branch November 30, 2023 03:02
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Report version when responding to NoOp Comand to match Sample App template
2 participants