-
Notifications
You must be signed in to change notification settings - Fork 2k
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
ztimer: add module removal information to doxygen and date it #18105
Conversation
85762d6
to
3795d07
Compare
b085ee5
to
a86c697
Compare
LGTM, please squash! @miri64 have your change requests been addressed? |
a86c697
to
b7fef93
Compare
* Will be removed after 2022.10 release. | ||
*/ | ||
/** | ||
* @typedef ztimer_now_t |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might not be needed, but not blocking for this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is easier to read this way (i think you suggest the @typedef ...
might not be needed)
missing an approval |
Contribution description
i missed the module in #18054
Testing procedure
Read the build artefact
Issues/PRs references
#18054