Skip to content

Conversation

jessebraham
Copy link
Member

Removes the direct boot image format. Also refactored away the ImageFormat trait since we only have one format now.

Copy link
Member

@SergioGasquez SergioGasquez left a comment

Choose a reason for hiding this comment

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

Changes LGTM! Just left a note regarding the test resources!

I've also done some testing, and I was able to flash all the targets but encountered #573 a few times, so it might be worth fixing it before the rc.

Copy link
Member

@MabezDev MabezDev left a comment

Choose a reason for hiding this comment

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

Whilst I can see a world where having it here is useful, I'm okay with removing it as we don't have any current use. We can always add it back in the future.

LGTM!

@jessebraham jessebraham merged commit 8814e83 into esp-rs:main Feb 6, 2024
@jessebraham jessebraham deleted the feature/remove-direct-boot branch February 6, 2024 18:03
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.

3 participants