Skip to content

Conversation

@crasbe
Copy link
Contributor

@crasbe crasbe commented May 7, 2025

Contribution description

During our seminar, we discovered some issues with various distributions (especially Fedora and Arch Linux, for the latter I added a commit in #21395).

Also, the Getting Started guide is outdated and said that it is not possible to pass through USB devices, which is not correct (anymore).

Furthermore, #21436 introduced a regression. The link to the WSL guide in the Getting Started page was dead.

All in all, I used the opportunity to clean the Getting Started documentation up a bit, even though it'll be eventually replaced by the Starlight documentation.

Testing procedure

Generate the documentation and see that everything still looks fine.

Issues/PRs references

Fix regression from #21436.

@crasbe crasbe requested review from aabadie and jia200x as code owners May 7, 2025 14:32
@crasbe crasbe added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels May 7, 2025
@github-actions github-actions bot added the Area: doc Area: Documentation label May 7, 2025
@riot-ci
Copy link

riot-ci commented May 7, 2025

Murdock results

✔️ PASSED

2e052cf doc/getting-started: fix links, minor cleanup

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

Artifacts

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.

Cool!

@Teufelchen1 Teufelchen1 added this pull request to the merge queue May 8, 2025
@crasbe crasbe force-pushed the pr/gettingstarted branch from e4843d8 to f03e9ec Compare May 8, 2025 12:17
@crasbe
Copy link
Contributor Author

crasbe commented May 8, 2025

I forgot to push the fixed reference links 👀

@crasbe crasbe removed this pull request from the merge queue due to a manual request May 8, 2025
@crasbe crasbe enabled auto-merge May 8, 2025 12:23
@crasbe crasbe added this pull request to the merge queue May 8, 2025
@crasbe crasbe removed this pull request from the merge queue due to a manual request May 8, 2025
@crasbe
Copy link
Contributor Author

crasbe commented May 8, 2025

@Teufelchen1 I just had a thought: wouldn't it be better to link to the new Guides page instead of linking to GitHub, which can only somewhat render the .mdx file?

Also ping @AnnsAnns.

@AnnsAnns
Copy link
Member

AnnsAnns commented May 8, 2025

I have nothing against that :D

@crasbe crasbe force-pushed the pr/gettingstarted branch from f03e9ec to 2e052cf Compare May 8, 2025 15:38
@Teufelchen1 Teufelchen1 added this pull request to the merge queue May 8, 2025
Merged via the queue into RIOT-OS:master with commit 5b5c1ba May 8, 2025
25 checks passed
@crasbe
Copy link
Contributor Author

crasbe commented May 8, 2025

Thanks for looking into this :)

@crasbe crasbe deleted the pr/gettingstarted branch May 8, 2025 18:40
@Teufelchen1 Teufelchen1 added this to the Release 2025.07 milestone Jul 14, 2025
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 Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants