Skip to content

Conversation

jessebraham
Copy link
Member

Closes #396 (I think..?)

@dragazo @MabezDev please confirm this fixes the issue, but as far as I can tell we just were never passing the command-line argument to the Flasher struct when it was provided.

@jessebraham jessebraham requested a review from SergioGasquez May 30, 2023 15:18
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.

LGTM, thanks for the fix!

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.

LGTM!

@jessebraham jessebraham merged commit 092a637 into esp-rs:main May 31, 2023
@jessebraham jessebraham deleted the fixes/flash-size branch May 31, 2023 15:11
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.

Flash size question

3 participants