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.

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

blobs: include documentation in doxygen #19200

Merged
merged 1 commit into from
Jan 26, 2023
Merged

blobs: include documentation in doxygen #19200

merged 1 commit into from
Jan 26, 2023

Conversation

bergzand
Copy link
Member

@bergzand bergzand commented Jan 25, 2023 •
edited
Loading

Contribution description

This PR changes the blob makefile documentation to be included in the doxygen output. I didn't slightly change the documentation itself to satisfy doxygen, just added another # and the doxygen boiler plating .

Testing procedure

Take a look at the docs

Issues/PRs references

None

@bergzand bergzand added Area: doc Area: Documentation Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Jan 25, 2023
@github-actions github-actions bot added the Area: build system Area: Build system label Jan 25, 2023
@riot-ci
Copy link

riot-ci commented Jan 25, 2023 •
edited
Loading

Murdock results

✔️ PASSED

96e37a4 blobs: include documentation in doxygen

Success Failures Total Runtime
1 0 1 48s

Artifacts

kaspar030
kaspar030 approved these changes Jan 25, 2023
View reviewed changes
Copy link
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

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

ACK.

@kaspar030
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Jan 26, 2023

Build succeeded:

@bors bors bot merged commit 7b75351 into RIOT-OS:master Jan 26, 2023
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.04 milestone Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kaspar030 kaspar030 kaspar030 approved these changes

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

Assignees
No one assigned
Labels
Area: build system Area: Build system Area: doc Area: Documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Milestone
Release 2023.04
Development

Successfully merging this pull request may close these issues.
4 participants
@bergzand @riot-ci @kaspar030 @MrKevinWeiss