Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Incorporate ST feedback #1844

Merged
merged 1 commit into from
Oct 5, 2022
Merged

chore: Incorporate ST feedback #1844

merged 1 commit into from
Oct 5, 2022

Conversation

zfields
Copy link
Contributor

@zfields zfields commented Oct 4, 2022

Clarify Swan LP1 interface defines.

We liked/appreciated your feedback, but we didn't get it changed before you merged.
Thank you for your quick responses, and good advice. ;-)

Copy link
Member

@fpistm fpistm left a comment

Choose a reason for hiding this comment

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

LGTM

@fpistm fpistm added this to In progress in STM32 core based on ST HAL via automation Oct 4, 2022
@fpistm fpistm added this to the 2.4.0 milestone Oct 4, 2022
@fpistm
Copy link
Member

fpistm commented Oct 4, 2022

We liked/appreciated your feedback, but we didn't get it changed before you merged.
Thank you for your quick responses, and good advice. ;-)

You didn't tell me you would make change 😛

@zfields
Copy link
Contributor Author

zfields commented Oct 5, 2022

I thought I was fast, but you were faster... 😝

This one is ready when you are. 👍

@fpistm fpistm merged commit ed4ef2f into stm32duino:main Oct 5, 2022
STM32 core based on ST HAL automation moved this from In progress to Done Oct 5, 2022
@zfields zfields deleted the swan_vcp branch February 28, 2023 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants