Skip to content

guides/c_tutorials/timers: Fix outdated intro#21673

Merged
crasbe merged 1 commit intoRIOT-OS:masterfrom
AnnsAnns:guide_fix_timers
Aug 20, 2025
Merged

guides/c_tutorials/timers: Fix outdated intro#21673
crasbe merged 1 commit intoRIOT-OS:masterfrom
AnnsAnns:guide_fix_timers

Conversation

@AnnsAnns
Copy link
Copy Markdown
Member

Contribution description

While going through the guides I noticed that the timers guide has a wrong intro that expects the user to have interacted with ztimer_sleep in the previous tutorial. At one point that was indeed correct but after a few reviews the ztimer_sleep was thrown out of the hello_world guide, thus not actually ever explaining how to add the module to the users Makefile.

Testing procedure

Issues/PRs references

@github-actions github-actions bot added the Area: doc Area: Documentation label Aug 20, 2025
@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 labels Aug 20, 2025
@AnnsAnns
Copy link
Copy Markdown
Member Author

fixed review comments 🐸

@AnnsAnns
Copy link
Copy Markdown
Member Author

AnnsAnns commented Aug 20, 2025

Applied 👍

@AnnsAnns
Copy link
Copy Markdown
Member Author

squashed in full confidence (that I probably have a whitespace error somewhere 😔)

@crasbe crasbe enabled auto-merge August 20, 2025 15:36
@riot-ci
Copy link
Copy Markdown

riot-ci commented Aug 20, 2025

Murdock results

✔️ PASSED

f7cd047 guides/c_tutorials/timers: Fix outdated intro

Success Failures Total Runtime
1 0 1 02m:38s

Artifacts

@crasbe crasbe added this pull request to the merge queue Aug 20, 2025
Merged via the queue into RIOT-OS:master with commit 0745ea5 Aug 20, 2025
26 checks passed
@AnnsAnns
Copy link
Copy Markdown
Member Author

ty

@benpicco benpicco added this to the Release 2025.10 milestone Dec 3, 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: 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