Skip to content

Conversation

@LeStarch
Copy link
Collaborator

Related Issue(s)
Has Unit Tests (y/n)
Documentation Included (y/n)

Change Description

Removes DEBUG_PRINT and other variants of this pattern from flight code.

Rationale

This is a step to addressing #1708.

Additionally, this pattern causes a lot of dead and commented-out code, which is not the best practice.

Testing/Review Recommendations

Please assert that only these changes have been made:

  1. DEBUG_PRINT lines were removed
  2. DEBUG_PRINT defines were removed
  3. Clauses and functions left empty by 1 or 2 were removed.

Future Work

Note any additional work that will be done relating to this issue.

@LeStarch LeStarch requested a review from thomas-bc January 22, 2025 22:51
Copy link
Collaborator

@thomas-bc thomas-bc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed 1), 2) and 3) from Review recommendations

@LeStarch LeStarch merged commit 706fadf into nasa:devel Jan 24, 2025
36 checks passed
@LeStarch LeStarch deleted the issue-1708-remove-debug-prints branch August 19, 2025 17:13
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.

2 participants