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 #50, Apply consistent Event ID names to common events #51

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

thnkslprpt
Copy link
Contributor

@thnkslprpt thnkslprpt commented Oct 21, 2022

Checklist

Describe the contribution

Testing performed
Only GitHub CI actions.

Expected behavior changes
No impact on code behavior (no logic changes).
Consistent Event ID names for the events which are common to all/most cFS components and apps will improve consistency and ease make code review/debugging easier.

Contributor Info
Avi Weiss @thnkslprpt

@dzbaker dzbaker added this to the Fornax milestone Nov 21, 2022
@dzbaker dzbaker modified the milestones: Fornax, Equuleus Dec 7, 2022
@thnkslprpt thnkslprpt force-pushed the fix-50-apply-consistent-event-id-names branch from 8ff4138 to 571cc29 Compare March 12, 2023 05:06
@thnkslprpt thnkslprpt force-pushed the fix-50-apply-consistent-event-id-names branch from 571cc29 to 91c0a4e Compare April 1, 2023 23:45
@thnkslprpt thnkslprpt force-pushed the fix-50-apply-consistent-event-id-names branch from 91c0a4e to a98c165 Compare November 1, 2023 17:42
@thnkslprpt thnkslprpt force-pushed the fix-50-apply-consistent-event-id-names branch from a98c165 to cc1834f Compare November 1, 2023 17:43
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.

Inconsistent Event ID naming
4 participants