tools: provide tool to create doxygen maintainer list #21062
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
miri64
added
the
Type: new feature
github-actions
bot
added
Area: doc
... rather than outdated wiki page.
miri64
force-pushed
the
tools/feat/maintainer-list
branch
from
187b795 to
4b31bfd
Compare
miri64
added
CI: skip compile test
That did not work. :-( @RIOT-OS/ci can you have a look? |
|
I don't know how that new html is created or where it ends up, but these lines deal with the preview: https://github.com/RIOT-OS/murdock-scripts/blob/931a4136b6efa7d44539826d665ae2b1561e16e1/build.sh#L281 |
jkarinkl
reviewed
Dec 6, 2024
miri64
added a commit
to miri64/RIOT
that referenced
this pull request
Dec 6, 2024
This is just the generated result from RIOT-OS#21062 while that PR is under development
miri64
added a commit
to miri64/RIOT
that referenced
this pull request
Dec 6, 2024
This is just the generated result from RIOT-OS#21062 while that PR is under development
github-actions
bot
added
the
Area: CI
miri64
added a commit
to miri64/RIOT
that referenced
this pull request
Dec 6, 2024
This is just the generated result from RIOT-OS#21062 while that PR is under development
miri64
force-pushed
the
tools/feat/maintainer-list
branch
from
eee5863 to
5fe56c7
Compare
miri64
force-pushed
the
tools/feat/maintainer-list
branch
from
5fe56c7 to
83b01c8
Compare
miri64
added a commit
to miri64/RIOT
that referenced
this pull request
Dec 6, 2024
This is just the generated result from RIOT-OS#21062 while that PR is under development
miri64
added a commit
to miri64/RIOT
that referenced
this pull request
Dec 6, 2024
This is just the generated result from RIOT-OS#21062 while that PR is under development
|
https://github.com/RIOT-OS/RIOT/actions/runs/12201211174/job/34039104721 is failing for this PR since the build was initiated from my fork (where the secret is not present). However, I did some testing on a feature branch. There it worked: https://github.com/RIOT-OS/RIOT/actions/runs/12200980590 |
|
Provided RIOT-OS/riot-os.org#141 which might be a better alternative |
|
Closing in favor of RIOT-OS/riot-os.org#141 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contribution description
This has been long discussed, but never set into action. Given that MAINTAINING.md now references a non-existent wiki page (which was outdated before its deletion), I say it is about time.
This basically takes the information from the RIOT-OS org teams and combines it with the information in CODEOWNERS. Caveat: One needs a token with read:org permission to get the teams.
I tried to match the CODEOWNERS patterns to doxygen groups somehow, but there either too little overlap (only @jia200x who chose documentation as an area of expertise had matching doxygen groups) or too much (e.g., @kaspar030 then received nearly every CPU, because he is interested in
pm.cwhich have an@ingroupfor each respective CPU or theirperiphinterface...), so I just gave up on that.Testing procedure
If all goes well, Murdock should just have a new
maintainers-list.htmlin its Documentation preview.Issues/PRs references
None so far.