Skip to content
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

applications: asset_tracker_v2: Use deferred logging v2 #5795

Merged
merged 1 commit into from
Oct 8, 2021

Conversation

simensrostad
Copy link
Contributor

Use deferred logging v2.

After the change to deferred logging, floats were not properly printed
anymore. This patch fixes this issue by using version 2 of deferred
logging.

Fix suggested in zephyrproject-rtos/zephyr#18351 and
documented in https://docs.zephyrproject.org/latest/reference/logging/index.html

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 6, 2021
@simensrostad simensrostad removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 6, 2021
@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

Use deferred logging v2.

After the change to deferred logging, floats were not properly printed
anymore. This patch fixes this issue by using version 2 of deferred
logging.

Fix suggested in
zephyrproject-rtos/zephyr#18351 and
documented in
https://docs.zephyrproject.org/latest/reference/logging/index.html

Signed-off-by: Simen S. Røstad <simen.rostad@nordicsemi.no>
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 7, 2021
@rlubos rlubos removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 8, 2021
@rlubos rlubos merged commit c808ef4 into nrfconnect:master Oct 8, 2021
@simensrostad simensrostad deleted the use-logger-v2 branch November 22, 2021 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants