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 #72, Update Reset Counters event type to INFORMATION #73

Merged

Conversation

thnkslprpt
Copy link
Contributor

Checklist

Describe the contribution

Testing performed
GitHub CI actions all passing successfully (incl. Build + Run, Unit/Functional Tests etc.).

Expected behavior changes
Reset Counters command event will be INFORMATION, aligning MD with the cFS standard.

System(s) tested on
Debian GNU/Linux 11 (bullseye)
Current main branch of cFS bundle.

Contributor Info
Avi Weiss   @thnkslprpt

@dzbaker
Copy link
Contributor

dzbaker commented Sep 26, 2024

Hi @thnkslprpt, apologies about the delay with this PR. I'd like to get it into the next round of PR reviews. Would you be able to resolve the merge conflict?

@thnkslprpt thnkslprpt force-pushed the fix-72-update-reset-counters-event-type branch from f4f103c to 0859eeb Compare September 27, 2024 05:49
@thnkslprpt
Copy link
Contributor Author

Hi @thnkslprpt, apologies about the delay with this PR. I'd like to get it into the next round of PR reviews. Would you be able to resolve the merge conflict?

@dzbaker No worries - updated and everything is green now

@dzbaker dzbaker merged commit c93e9d5 into nasa:main Oct 3, 2024
16 checks passed
@thnkslprpt thnkslprpt deleted the fix-72-update-reset-counters-event-type branch October 3, 2024 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Standardization of Command Responses
3 participants