Skip to content

Bluetooth: Doc: Update doxygen groups #88969

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Bluetooth: Doc: Update doxygen groups #88969

wants to merge 1 commit into from

Conversation

omkar3141
Copy link
Collaborator

Move top most level Bluetooth subsystem related APIs (enable, disable, etc.) under 'bluetooth' group, and rest under 'bt_gap' group.

@github-actions github-actions bot added area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth labels Apr 23, 2025
@omkar3141 omkar3141 requested a review from Copilot April 23, 2025 13:36
Copilot
Copilot AI reviewed Apr 23, 2025
View reviewed changes
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 wasn't able to review any files in this pull request.

Files not reviewed (1)
  • include/zephyr/bluetooth/bluetooth.h: Language not supported

@omkar3141 omkar3141 force-pushed the omku/bt_doc_group_update branch from 8bef32c to ae466ef Compare April 23, 2025 13:47
@omkar3141
Bluetooth: Doc: Update doxygen groups
b0bcf53 
Move top most level Bluetooth subsystem related APIs (enable, disable,
etc.) under 'bluetooth' group, and rest under 'bt_gap' group.

Signed-off-by: Omkar Kulkarni <[email protected]>
@omkar3141 omkar3141 force-pushed the omku/bt_doc_group_update branch from ae466ef to b0bcf53 Compare April 23, 2025 15:36
@jhedberg jhedberg added the DNM This PR should not be merged (Do Not Merge) label Apr 23, 2025
@jhedberg jhedberg changed the title DNM - Bluetooth: Doc: Update doxygen groups Bluetooth: Doc: Update doxygen groups Apr 23, 2025
@jhedberg
Copy link
Member

@omkar3141 please use the "DNM" label to indicate that a PR shouldn't be merged (yet). I updated this PR accordingly.

@jhedberg
Copy link
Member

@omkar3141 please use the "DNM" label to indicate that a PR shouldn't be merged (yet). I updated this PR accordingly.

Alternatively, keep the PR as Draft until you consider it to be ready.

@omkar3141
Copy link
Collaborator Author

@omkar3141 please use the "DNM" label to indicate that a PR shouldn't be merged (yet). I updated this PR accordingly.

Alternatively, keep the PR as Draft until you consider it to be ready.

I need to trigger doc build to see how this is looking.

@Thalley
Copy link
Collaborator

Thalley commented Apr 24, 2025

@omkar3141 please use the "DNM" label to indicate that a PR shouldn't be merged (yet). I updated this PR accordingly.

Alternatively, keep the PR as Draft until you consider it to be ready.

I need to trigger doc build to see how this is looking.

CI is still run on draft PRs :)

@omkar3141 omkar3141 marked this pull request as draft April 24, 2025 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@alwa-nordic alwa-nordic Awaiting requested review from alwa-nordic

@cvinayak cvinayak Awaiting requested review from cvinayak

@hermabe hermabe Awaiting requested review from hermabe

@jhedberg jhedberg Awaiting requested review from jhedberg

@rugeGerritsen rugeGerritsen Awaiting requested review from rugeGerritsen

@sjanc sjanc Awaiting requested review from sjanc

@Thalley Thalley Awaiting requested review from Thalley

@theob-pro theob-pro Awaiting requested review from theob-pro

At least 2 approving reviews are required to merge this pull request.

Assignees

@jhedberg jhedberg

@alwa-nordic alwa-nordic

Labels
area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth DNM This PR should not be merged (Do Not Merge)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@omkar3141 @jhedberg @Thalley @alwa-nordic