-
Notifications
You must be signed in to change notification settings - Fork 29
lib: event_scheduler: rename to bm_scheduler #491
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
Conversation
|
You can find the documentation preview for this PR here. |
da84c7f to
c93b253
Compare
Rename to bm_scheduler to follow the repository new naming scheme. Signed-off-by: Emanuele Di Santo <[email protected]>
|
|
||
| module=EVENT_SCHEDULER | ||
| module=BM_SCHEDULER | ||
| module-str=Event scheduler |
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.
Shouldn't this be renamed too? Same for the prompt at the top
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.
Maybe, timer and buttons say: "Buttons library" and "Timer library"
The module-str instead says "Bare metal buttons" and "Bare metal timer"
Updates to the documentation, call the library "Bare Metal scheduler". Signed-off-by: Emanuele Di Santo <[email protected]>
|
Rename from generic evt_handler_t to bm_scheduler_fn_t. Signed-off-by: Emanuele Di Santo <[email protected]>
|
@nrfconnect/ncs-co-build-system , @nrfconnect/ncs-code-owners |



Rename to
bm_schedulerto follow the repository new naming scheme.