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

doccheck: make the file pattern more match riot.doxyfile #18431

Merged
merged 2 commits into from
Sep 19, 2022
Merged

doccheck: make the file pattern more match riot.doxyfile #18431

merged 2 commits into from
Sep 19, 2022

Conversation

kfessel
Copy link
Contributor

@kfessel kfessel commented Aug 10, 2022 •
edited
Loading

Contribution description

doccheck checks the @defgroup and @ingroup commands for doxygen sadly it uses a wrong list of files
this brings them more in sync

the file pattern used by doxygen can be found here:

RIOT/doc/doxygen/riot.doxyfile

Lines 801 to 804 in 2dd1854

FILE_PATTERNS = *.txt \
*.h \
*.hpp

Testing procedure

run doccheck

Issues/PRs references

@github-actions github-actions bot added Area: sys Area: System Area: timers Area: timer subsystems Area: tools Area: Supplementary tools labels Aug 10, 2022
@kfessel kfessel mentioned this pull request Aug 10, 2022
Merged
@kfessel kfessel added 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 Aug 10, 2022
@kfessel kfessel requested a review from miri64 August 15, 2022 12:43
@kfessel
Copy link
Contributor Author

kfessel commented Sep 15, 2022

maybe @miri64 likes to have a look at this

@kfessel kfessel merged commit e43a75d into RIOT-OS:master Sep 19, 2022
@maribu maribu added this to the Release 2022.10 milestone Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maribu maribu maribu approved these changes

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

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

@miri64 miri64 Awaiting requested review from miri64

Assignees
No one assigned
Labels
Area: sys Area: System Area: timers Area: timer subsystems Area: tools Area: Supplementary tools 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
Projects
None yet
Milestone
Release 2022.10
Development

Successfully merging this pull request may close these issues.
2 participants
@kfessel @maribu