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: Update sdk-zephyr with SPI test for error cases #14876

Closed
wants to merge 1 commit into from
Closed

manifest: Update sdk-zephyr with SPI test for error cases #14876

wants to merge 1 commit into from

Conversation

nordic-segl
Copy link
Contributor

Include changes from:
nrfconnect/sdk-zephyr#1637

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

NordicBuilder commented Apr 16, 2024 •
edited
Loading

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@3733e70 nrfconnect/sdk-zephyr@6c515dd nrfconnect/[email protected]

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 16, 2024 •
edited
Loading

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
desktop52_verification X
test-fw-nrfconnect-chip X
test-fw-nrfconnect-fem X
test-fw-nrfconnect-nfc X
test-fw-nrfconnect-nrf-iot_cloud X
test-fw-nrfconnect-nrf-iot_mosh X
test-fw-nrfconnect-nrf-iot_positioning X
test-fw-nrfconnect-nrf-iot_thingy91 X
test-fw-nrfconnect-nrf-iot_zephyr_lwm2m X
test-fw-nrfconnect-rpc X
test-fw-nrfconnect-rs X
test-fw-nrfconnect-thread X
test-low-level X
test-sdk-find-my X
test-sdk-wifi X

Detailed information of selected test modules

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

@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.

@nordic-segl nordic-segl force-pushed the NRFX-5624_Extend-test-coverage-for-spi-driver branch from 58bb6c6 to 3082368 Compare April 16, 2024 09:17
@nordic-segl
Copy link
Contributor Author

Requires
nrfconnect/sdk-zephyr#1598
to be merged first.

@nordic-segl nordic-segl force-pushed the NRFX-5624_Extend-test-coverage-for-spi-driver branch from 3082368 to 173a3c2 Compare April 17, 2024 06:56
@nordic-segl
Copy link
Contributor Author

[rebased on top of main - c12599e]

@nordic-segl nordic-segl force-pushed the NRFX-5624_Extend-test-coverage-for-spi-driver branch 5 times, most recently from a0e113b to f941d65 Compare April 23, 2024 06:26
@nordic-piks nordic-piks added this to the 2.6.99-dev1 milestone Apr 24, 2024
@nordic-segl nordic-segl force-pushed the NRFX-5624_Extend-test-coverage-for-spi-driver branch from f941d65 to 1407d63 Compare April 24, 2024 12:56
@NordicBuilder NordicBuilder removed the DNM label Apr 24, 2024
@nordic-segl nordic-segl force-pushed the NRFX-5624_Extend-test-coverage-for-spi-driver branch from 1407d63 to a20b479 Compare April 25, 2024 06:24
@cvinayak
Copy link
Contributor

@nordic-segl rebase resolve conflict and push, github will close this PR as merged if this branch has not change with main....

@nordic-segl
Copy link
Contributor Author

nordic-segl commented Apr 25, 2024 •
edited
Loading

Already included in the Manifest.
https://github.com/nrfconnect/sdk-zephyr/commits/main/
This PR switches sdk-zephyr to 6c515dd which is behind f50ad41 (in the manifest).

@nordic-segl nordic-segl deleted the NRFX-5624_Extend-test-coverage-for-spi-driver branch April 25, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nordic-piks nordic-piks nordic-piks approved these changes

@nordic-pikr nordic-pikr nordic-pikr approved these changes

@nordic-babu nordic-babu Awaiting requested review from nordic-babu

@nordic-bami nordic-bami Awaiting requested review from nordic-bami

@carlescufi carlescufi Awaiting requested review from carlescufi

@tejlmand tejlmand Awaiting requested review from tejlmand

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

Successfully merging this pull request may close these issues.
5 participants
@nordic-segl @NordicBuilder @cvinayak @nordic-piks @nordic-pikr