Skip to content

Simplify Avionics Shell (UARTCommandHandler)#77

Merged
Elan456 merged 6 commits intomainfrom
chore/clean-ups
Feb 19, 2026
Merged

Simplify Avionics Shell (UARTCommandHandler)#77
Elan456 merged 6 commits intomainfrom
chore/clean-ups

Conversation

@Elan456
Copy link
Copy Markdown
Contributor

@Elan456 Elan456 commented Feb 18, 2026

Description

Summary of Changes

  • Add more testing hooks for analog voltage reading in HAL
  • Removal of command history in Avionics Shell
  • Modularization of input handling in the Avionics Shell

Motivation

Make the codebase easier to maintain, test, and extend, while removing unused or overly complex features.
The UARTCommandHandler was utilizing dynamic memory for the command history, so it's good to eliminate that until it can be re-implemented with a static allocation.


Testing

Tested by opening a shell on MARTHA 1.4 and ensuring most things work right

Check all that apply:

  • I have added or modified 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:

    (Provide reasoning here)


Building

  • This change successfully builds in at least one of the following environments:
    • Native repo
    • Target device(s) (e.g., MARTHA)
  • If the build fails in any environment, explain why:

    (Provide reasoning here)


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).

@Elan456 Elan456 requested a review from Jshepherd06 February 19, 2026 17:26
Copy link
Copy Markdown
Contributor

@Jshepherd06 Jshepherd06 left a comment

Choose a reason for hiding this comment

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

looks good

@Elan456 Elan456 merged commit be1632c into main Feb 19, 2026
2 of 3 checks passed
@Elan456 Elan456 deleted the chore/clean-ups branch February 19, 2026 22:36
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