Skip to content

doccheck: less exclude_simple - #22513

Open
kfessel wants to merge 3 commits into
RIOT-OS:masterfrom
kfessel:p-doccheck-less-simple
Open

doccheck: less exclude_simple#22513
kfessel wants to merge 3 commits into
RIOT-OS:masterfrom
kfessel:p-doccheck-less-simple

Conversation

@kfessel

@kfessel kfessel commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Contribution description

reduce the number of warning excluded from reporting without recreating the entire file

probably a bunch of them just got fixed over time

some of them might no longer be reported due to documentation not build

steps to reproduce the original PR:

#in dist/tools/dockcheck

mv exclude_simple exclude_simple.o
sort exclude_simple.o| uniq - exclude_simple.o.u 
touch exclude_simple

./check.sh > exclude_simple.new
#remove filenames ( start print at "warning:)
sed -E -e "s#^.*(warning:)#\1#" <exclude_simple.new >exclude_simple.n

sort exclude_simple.n| uniq - exclude_simple.n.u
diff -U 0 exclude_simple.o.u exclude_simple.n > exclude_simple.diff
# next awk line was recreated using duck.ai after the original PR to replace
# the use of https://kfessel.github.io/JS-helpers/regexrunner.html
awk '/^-/ { print substr($0, 2) }' exclude_simple.diff > exclude_simple.filter 

grep -Fvf exclude_simple.filter exclude_simple.o > exclude_simple

Testing procedure

lets see what the ci says since this file is mostly for ci entertainment that should be good enough

maybe one likes to check if the documentation not build is something that plays into this

A AI review might focus on documentation not build

Issues/PRs references

#22512
#22327
#19240

Declaration of AI-Tools / LLMs usage:

AI-Tools / LLMs that were used are:

  • none

@github-actions github-actions Bot added the Area: tools Area: Supplementary tools label Jul 28, 2026
@crasbe crasbe added 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 AI: Not Used AI was stated to not be used in this PR/Issue labels Jul 28, 2026
@riot-ci

riot-ci commented Jul 28, 2026

Copy link
Copy Markdown

Murdock results

✔️ PASSED

ae5b25b f: add some back

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

Artifacts

@kfessel
kfessel requested a review from Copilot July 28, 2026 12:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kfessel
kfessel marked this pull request as ready for review July 28, 2026 14:21

@crasbe crasbe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for cleaning this up, please squash :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI: Not Used AI was stated to not be used in this PR/Issue 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 Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants