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

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-26-apply-consistent-event-id-names branch from 890e59c to bee2f62 Compare March 12, 2023 03:48
@havencarlson
Copy link
Contributor

@thnkslprpt Can you fix the merge conflicts for this pull request?

@thnkslprpt thnkslprpt force-pushed the fix-26-apply-consistent-event-id-names branch from bee2f62 to 7d51d63 Compare March 30, 2023 21:13
@thnkslprpt thnkslprpt force-pushed the fix-26-apply-consistent-event-id-names branch from 7d51d63 to 1f7715a Compare March 30, 2023 21:15
@thnkslprpt
Copy link
Contributor Author

@thnkslprpt Can you fix the merge conflicts for this pull request?

I think that's all cleared up now mate.
@havencarlson

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