Skip to content

doc: reorder some drivers#21943

Merged
AnnsAnns merged 9 commits intoRIOT-OS:masterfrom
mguetschow:doc-drivers
Dec 9, 2025
Merged

doc: reorder some drivers#21943
AnnsAnns merged 9 commits intoRIOT-OS:masterfrom
mguetschow:doc-drivers

Conversation

@mguetschow
Copy link
Copy Markdown
Contributor

Contribution description

While skimming through https://doc.riot-os.org/group__drivers.html I noticed that some drivers could be grouped in a better way.

Testing procedure

Check CI-generated doc.

@mguetschow mguetschow requested a review from jia200x as a code owner December 9, 2025 08:48
@github-actions github-actions bot added Area: network Area: Networking Area: doc Area: Documentation Area: pkg Area: External package ports Area: drivers Area: Device drivers Area: sys Area: System labels Dec 9, 2025
@github-actions github-actions bot added Platform: native Platform: This PR/issue effects the native platform Area: cpu Area: CPU/MCU ports labels Dec 9, 2025
@github-actions github-actions bot added the Platform: ARM Platform: This PR/issue effects ARM-based platforms label Dec 9, 2025
@mguetschow mguetschow added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Dec 9, 2025
@riot-ci
Copy link
Copy Markdown

riot-ci commented Dec 9, 2025

Murdock results

✔️ PASSED

e5e54a6 pkg/cryptoauthlib: md codeblocks in doc.md

Success Failures Total Runtime
1 0 1 01m:23s

Artifacts

@crasbe crasbe added the Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation label Dec 9, 2025
Copy link
Copy Markdown
Member

@AnnsAnns AnnsAnns left a comment

Choose a reason for hiding this comment

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

Ty for this, doxygen looks fine, just a minor nitpick

Copy link
Copy Markdown
Member

@AnnsAnns AnnsAnns left a comment

Choose a reason for hiding this comment

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

🎾 Squash

@AnnsAnns AnnsAnns enabled auto-merge December 9, 2025 14:20
@AnnsAnns AnnsAnns disabled auto-merge December 9, 2025 14:20
@mguetschow
Copy link
Copy Markdown
Contributor Author

🎾 Squash

Should I? They are in principle all individual commits targeting different files. E.g. I would leave the code-block change to a separate commit so it is easier to see the diff compared to when it would be intermixed with the * removals from the doc.txt to doc.md conversion.

@AnnsAnns
Copy link
Copy Markdown
Member

AnnsAnns commented Dec 9, 2025

I will be real with you, I instinctively said that 😜. I'm fine with the commits being split up.

@AnnsAnns AnnsAnns added this pull request to the merge queue Dec 9, 2025
Merged via the queue into RIOT-OS:master with commit 5b38177 Dec 9, 2025
27 checks passed
@mguetschow
Copy link
Copy Markdown
Contributor Author

Thank you! :)

@mguetschow mguetschow deleted the doc-drivers branch December 10, 2025 09:24
@leandrolanzieri leandrolanzieri added this to the Release 2026.01 milestone Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: cpu Area: CPU/MCU ports Area: doc Area: Documentation Area: drivers Area: Device drivers Area: network Area: Networking Area: pkg Area: External package ports Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: native Platform: This PR/issue effects the native platform Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants