X Tutup
Skip to content

feat(box-emu): Refactor to reduce deps on EspBox, instead depending on box-emu#96

Merged
finger563 merged 1 commit intomainfrom
feat/esp-box-refactor
Mar 20, 2025
Merged

feat(box-emu): Refactor to reduce deps on EspBox, instead depending on box-emu#96
finger563 merged 1 commit intomainfrom
feat/esp-box-refactor

Conversation

@finger563
Copy link
Contributor

Description

  • Update box-emu component to include wrappers / accessors for some underlying audio and video functions from the EspBox
  • Update all other affected components to replace access of EspBox:: with BoxEmu:: wrappers, so that it would be easier / simpler to target other systems

Motivation and Context

WIP to support #95

How has this been tested?

Ensure the project still builds

Screenshots (if appropriate, e.g. schematic, board, console logs, lab pictures):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update
  • Hardware (schematic, board, system design) change
  • Software change

Checklist:

  • My change requires a change to the documentation.
  • I have added / updated the documentation related to this change via either README or WIKI

Software

  • I have added tests to cover my changes.
  • I have updated the .github/workflows/build.yml file to add my new test to the automated cloud build github action.
  • All new and existing tests passed.
  • My code follows the code style of this project.

…n box-emu

* Update `box-emu` component to include wrappers / accessors for some underlying audio and video functions from the EspBox
* Update all other affected components to replace access of `EspBox::` with `BoxEmu::` wrappers, so that it would be easier / simpler to target other systems

WIP to support #95

Ensure the project still builds
@finger563 finger563 self-assigned this Mar 20, 2025
@finger563 finger563 added enhancement New feature or request box-emu labels Mar 20, 2025
@github-actions
Copy link

✅Static analysis result - no issues found! ✅

@finger563 finger563 merged commit 0070d1c into main Mar 20, 2025
3 checks passed
@finger563 finger563 deleted the feat/esp-box-refactor branch March 20, 2025 03:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

box-emu enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

X Tutup