Skip to content

Conversation

DeJusten
Copy link
Contributor

Contribution description

In the description the compiler switch -D is missing for setting the corresponding macro

Testing procedure

Only documentation, no test necessary

Issues/PRs references

No Issue available

@github-actions github-actions bot added Area: doc Area: Documentation Area: sys Area: System labels Feb 18, 2025
Copy link
Contributor

@mguetschow mguetschow left a comment

Choose a reason for hiding this comment

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

Thanks for finding and fixing!

@mguetschow mguetschow added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 19, 2025
@riot-ci
Copy link

riot-ci commented Feb 19, 2025

Murdock results

✔️ PASSED

e7e39a6 sys/malloc_monitor: Fix documentation

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

Artifacts

@miri64
Copy link
Member

miri64 commented Feb 19, 2025

Commit message still needs to follow the common format. Which doc.txt? What does the update entail?

@maribu maribu changed the title Update doc.txt sys/malloc_monitor: Fix documentation Feb 20, 2025
In the description the compiler switch -D is missing for setting the
corresponding macro.
@maribu
Copy link
Member

maribu commented Mar 14, 2025

I was so free to change the commit message as requested here:

Commit message still needs to follow the common format. Which doc.txt? What does the update entail?

@maribu maribu enabled auto-merge March 14, 2025 12:18
@maribu maribu added this pull request to the merge queue Mar 14, 2025
Merged via the queue into RIOT-OS:master with commit f54d012 Mar 14, 2025
25 checks passed
@mguetschow mguetschow added this to the Release 2025.04 milestone Apr 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: doc Area: Documentation Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants