Skip to content

guides: use absolute path to fix url #21571

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

Merged
merged 1 commit into from
Jul 1, 2025

Conversation

mguetschow
Copy link
Contributor

Contribution description

The link in the box on https://guide.riot-os.org/getting-started/install-wsl/ is broken.

Testing procedure

Check the guides preview (do we have that?).

Issues/PRs references

Also, I'm a bit surprised to find the WSL installation guide above the general "Setup Development Environment" in the navigation tree, is that intentional @AnnsAnns ?

@github-actions github-actions bot added the Area: doc Area: Documentation label Jun 30, 2025
@AnnsAnns
Copy link
Contributor

Yes this is intentional since its basically a Step -1 for people on Windows 😄

@crasbe crasbe added Type: enhancement The issue suggests enhanceable parts / The PR enhances 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 Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) and removed Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation labels Jun 30, 2025
@riot-ci
Copy link

riot-ci commented Jun 30, 2025

Murdock results

✔️ PASSED

9b185b3 guides: use absolute path to fix url

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

Artifacts

@mguetschow mguetschow requested a review from crasbe July 1, 2025 14:37
Copy link
Contributor

@crasbe crasbe left a comment

Choose a reason for hiding this comment

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

Tried it out locally and the old version does not work, the new version works.

@crasbe crasbe enabled auto-merge July 1, 2025 23:13
@crasbe crasbe added this pull request to the merge queue Jul 1, 2025
Merged via the queue into RIOT-OS:master with commit f7bda59 Jul 1, 2025
31 checks passed
@mguetschow mguetschow deleted the guides-url-typo branch July 2, 2025 08:11
@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 CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants