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 #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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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