Skip to content

Added CURE prefix to data names - #102

Open
EduardoS10 wants to merge 1 commit into
mainfrom
issue-37-cure
Open

Added CURE prefix to data names#102
EduardoS10 wants to merge 1 commit into
mainfrom
issue-37-cure

Conversation

@EduardoS10

Copy link
Copy Markdown

Description

Summary of Changes

  • Added the CURE_ prefix to all data names defined in DataNames.h.
  • Updated affected source files to use the renamed data names.
  • Updated telemetry documentation to match the new naming.

Motivation

  • Prevents naming conflicts with sensor drivers and other systems.
  • Keeps data labels clearly scoped to CURE.

Closes #37


Testing

Check all that apply:

  • I have added or modified unit tests to cover these changes.
  • I have manually tested the changes on the target device(s) or environment(s).
  • Existing tests were reviewed to ensure they still work as expected.
  • If tests were not added or modified, explain why:

    This change only renames existing data identifiers while preserving their values and behavior. A native build was attempted; it still fails on pre-existing compiler warnings/errors unrelated to this change.


Checklist

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code where necessary for clarity.
  • I have made corresponding updates to documentation (if applicable).

@EduardoS10

Copy link
Copy Markdown
Author

The device build is failing because MARTHA-1.4 still references the old DataNames without the CURE_ prefix. The Avionics unit tests and clang-tidy checks are passing. Should I update the MARTHA references in a separate PR as well?

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.

Replace all DataNames with a CURE_ prefix

1 participant