Skip to content

tests: Bluetooth: Tester: Cleanup includes #88500

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

Merged
merged 2 commits into from
Apr 24, 2025

Conversation

Thalley
Copy link
Collaborator

@Thalley Thalley commented Apr 11, 2025

Update the includes so that only what is actually used is included, and so that everything is used is includes (IWYU).

Thalley added 2 commits April 11, 2025 15:30
Update the includes so that only what is actually used is included,
and so that everything is used is includes (IWYU).

Signed-off-by: Emil Gydesen <[email protected]>
The va.h file used both uintX and bool without including
the appropriate header files, which can cause a compile issue
if the file is included in the wrong order.

Signed-off-by: Emil Gydesen <[email protected]>
@Thalley Thalley requested a review from Copilot April 11, 2025 13:46
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 19 out of 19 changed files in this pull request and generated no comments.

@Thalley Thalley marked this pull request as ready for review April 11, 2025 21:08
@github-actions github-actions bot added area: Bluetooth area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests area: Bluetooth Mesh labels Apr 11, 2025
@kartben kartben merged commit c8742a3 into zephyrproject-rtos:main Apr 24, 2025
28 checks passed
@Thalley Thalley deleted the bttester_include_fixes branch April 24, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Mesh area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests area: Bluetooth
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants