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

[BT] fixes (IDFGH-8253) #9741

Closed
wants to merge 1 commit into from
Closed

Conversation

tgotic
Copy link
Contributor

@tgotic tgotic commented Sep 7, 2022

A set of small fixes on Bluetooth.

A set of small fixes on Bluetooth.
@espressif-bot espressif-bot added the Status: Opened Issue is new label Sep 7, 2022
@github-actions github-actions bot changed the title [BT] fixes [BT] fixes (IDFGH-8253) Sep 7, 2022
@Alvin1Zhang
Copy link
Collaborator

Thanks for your contribution.

@CLAassistant
Copy link

CLAassistant commented Apr 20, 2023

CLA assistant check
All committers have signed the CLA.

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: Won't Do This will not be worked on and removed Status: Opened Issue is new labels Jul 11, 2023
@esp-lrh
Copy link
Collaborator

esp-lrh commented Nov 7, 2023

Hi, @tgotic, thanks for you contribution, this kind of coding style issues are indeed prevalent in the Bluedroid codebase, and it's challenging to thoroughly identify and address all of them. Making only a small subset of fixes wouldn't result in substantial improvements. Therefore, we've decided not to merge this pull request.

Thank you again for your contribution. Our team will work on addressing these coding style issues collectively in the future.

@Alvin1Zhang
Copy link
Collaborator

Thanks for contribution again.

@Alvin1Zhang Alvin1Zhang closed this Nov 7, 2023
@tgotic tgotic deleted the bt_fixes_take3 branch November 9, 2023 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Won't Do This will not be worked on Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants