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.

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

MAINTAINING.md/doc: move links from wiki to doc #21066

Merged
merged 9 commits into from
Dec 6, 2024

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Dec 6, 2024

Contribution description

There are some (even outdated) links to the wiki from our MAINTAINING.md. This fixes that.

Testing procedure

Read. Some links might not work yet, as they require doc.riot-os.org to be rendered in master. Just confirm that the .html name is the same as in the Murdock preview.

Issues/PRs references

None

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

riot-ci commented Dec 6, 2024

Murdock results

✔️ PASSED

32cdf7e MAINTAINING.md: move links from wiki to doc

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

Artifacts

@miri64 miri64 force-pushed the doc/enh/maintaining-updates branch from a064935 to 93f210c Compare December 6, 2024 12:50
@miri64 miri64 changed the title MAINTAINING.md/doc: move links from wiki to doc @miri64 MAINTAINING.md/doc: move links from wiki to doc Dec 6, 2024
Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

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

Thx for cleaning this up :)

@miri64 miri64 added this pull request to the merge queue Dec 6, 2024
github-merge-queue bot pushed a commit that referenced this pull request Dec 6, 2024
 MAINTAINING.md/doc: move links from wiki to doc
@miri64 miri64 removed this pull request from the merge queue due to a manual request Dec 6, 2024
@miri64
Copy link
Member Author

miri64 commented Dec 6, 2024

Since the changes to the MAINTAINING.md are only moving some links around, do we need a second ACK? (I guess by asking this question I ask for a second ACK anyways 😅)

@miri64
Copy link
Member Author

miri64 commented Dec 6, 2024

Apparently the Doxygen version on Murdock does not like links in titles, so I fixed the maintainers list (in #21062 as well) to put the Github profile link within the body of the text.

@miri64 miri64 force-pushed the doc/enh/maintaining-updates branch from a23c7e0 to b14a078 Compare December 6, 2024 14:09
This is just the generated result from RIOT-OS#21062 while that PR is under
development
@miri64 miri64 force-pushed the doc/enh/maintaining-updates branch from b14a078 to 32cdf7e Compare December 6, 2024 14:16
Copy link
Contributor

@Teufelchen1 Teufelchen1 left a comment

Choose a reason for hiding this comment

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

Nothing to complain 👍

@miri64 miri64 added this pull request to the merge queue Dec 6, 2024
Merged via the queue into RIOT-OS:master with commit cb5500b Dec 6, 2024
26 checks passed
@miri64 miri64 deleted the doc/enh/maintaining-updates branch December 6, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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
Development

Successfully merging this pull request may close these issues.

4 participants