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.

Sign up for GitHub

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

Jump to bottom

manifest: openthread: update to \dc6bf36\ #13211

Merged
merged 1 commit into from
Nov 23, 2023
Merged

manifest: openthread: update to \dc6bf36\ #13211

merged 1 commit into from
Nov 23, 2023

Conversation

NordicBuilder
Copy link
Contributor

Regular OpenThread update to `dc6bf36`

Signed-off-by: Nordic Builder pylon@nordicsemi.no

@NordicBuilder
manifest: openthread: update to dc6bf36
2c2e980 
Regular OpenThread update to `dc6bf36`

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
@NordicBuilder
Copy link
Contributor Author

List of modified OpenThread API files :
etc/cmake/options.cmake
include/openthread/platform/
include/openthread/platform/radio.h
include/openthread/platform/toolchain.h

@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest labels Nov 23, 2023
@NordicBuilder
Copy link
Contributor Author

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
openthread nrfconnect/sdk-openthread@a05954b nrfconnect/sdk-openthread@dc6bf36 (main) nrfconnect/sdk-openthread@a05954b0..dc6bf363

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor Author

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-thread X

test-fw-nrfconnect-thread: added because there was no .github/test-spec.yml in 'openthread'

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@edmont
Copy link
Contributor

edmont commented Nov 23, 2023

Triggered tasks:

@maciejbaczmanski
Copy link
Member

Triggered tasks:

thanks @edmont , I'll handle both tasks tomorrow

edmont
edmont approved these changes Nov 23, 2023
View reviewed changes
Copy link
Contributor

@edmont edmont left a comment

Choose a reason for hiding this comment

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

Thanks @maciejbaczmanski. Not blockers for merging this PR though.

@cvinayak cvinayak merged commit d1fd7e8 into nrfconnect:main Nov 23, 2023
17 checks passed
@NordicBuilder NordicBuilder deleted the pr/update-openthread branch November 29, 2023 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maciejbaczmanski maciejbaczmanski maciejbaczmanski approved these changes

@edmont edmont edmont approved these changes

@carlescufi carlescufi Awaiting requested review from carlescufi

@tejlmand tejlmand Awaiting requested review from tejlmand

@canisLupus1313 canisLupus1313 Awaiting requested review from canisLupus1313

@mariuszpos mariuszpos Awaiting requested review from mariuszpos

@pjasinski990 pjasinski990 Awaiting requested review from pjasinski990

Assignees
No one assigned
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-openthread
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@NordicBuilder @edmont @maciejbaczmanski @cvinayak