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

Fast Pair headers: doxygen dot alignment #15754

Merged
merged 2 commits into from
Jun 6, 2024

Conversation

kapi-no
Copy link
Contributor

@kapi-no kapi-no commented Jun 5, 2024

No description provided.

Fixed missing dots in the Fast Pair FMDN header to make the doxygen
documentation more consistent and to align it with the main Fast Pair
header.

Signed-off-by: Kamil Piszczek <[email protected]>
Fixed missing dots in the Fast Pair UUID header to make the doxygen
documentation more consistent and to align it with the main Fast Pair
header.

Signed-off-by: Kamil Piszczek <[email protected]>
@kapi-no kapi-no requested review from divipillai and annwoj June 5, 2024 13:02
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jun 5, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jun 5, 2024

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
desktop52_verification X
test-fw-nrfconnect-ble X
test-fw-nrfconnect-ble_samples X
test-sdk-find-my X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@kapi-no
Copy link
Contributor Author

kapi-no commented Jun 6, 2024

The BLE integration fails at the gitlinter stage but the tests pass for this PR. See this comment:

#15634 (comment)

Also on the test_ble_host stage, but this PR just aligns the docs.

@kapi-no kapi-no removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jun 6, 2024
@rlubos rlubos merged commit b58886a into nrfconnect:main Jun 6, 2024
19 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants