-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
doc: remove breathe from all docsets #17251
base: main
Are you sure you want to change the base?
Conversation
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: f19d4502e1d7ad0fdff69f78b7a532df76ead3df more detailssdk-nrf:
nrfxlib:
Github labels
List of changed files detected by CI (287)
Outputs:ToolchainVersion: 2aae60c2f9 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
3c6425b
to
5d95612
Compare
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. |
098430f
to
9f8898f
Compare
... until post-Breathe era becomes a reality. Ref: nrfconnect#17251 Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
until post-Breathe era becomes a reality. Ref: nrfconnect#17251 Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
until post-Breathe era becomes a reality. Ref: nrfconnect#17251 Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
until post-Breathe era becomes a reality. Ref: nrfconnect#17251 Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
52f08fb
to
bc6c8ad
Compare
`void` functions do not return anything, so they can't have Doxygen documentation. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
This is required to render Doxygen content in the same Sphinx output folder, as Doxygen is run first. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
So that further Sphinx+Doxygen docsets are easy to add. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Native Doxygen content will be served, and Sphinx will link to it. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
It's no longer required as we're not using breathe anymore. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Breathe will no longer be used. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
It is no longer needed after breathe removal. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
We are now serving Doxygen content, so let's drop the redundant Sphinx "shim". Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
bc6c8ad
to
f19d450
Compare
until post-Breathe era becomes a reality. Ref: nrfconnect#17251 Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
until post-Breathe era becomes a reality. Ref: nrfconnect#17251 Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
No description provided.