Skip to content

Conversation

@kapbh
Copy link
Contributor

@kapbh kapbh commented Apr 15, 2025

Wi-Fi cred shell has been renamed from wifi_cred to wifi cred.

SHEL-3589

Wi-Fi cred shell has been renamed from wifi_cred to wifi cred.

SHEL-3589

Signed-off-by: Kapil Bhatt <[email protected]>
@kapbh kapbh requested review from a team as code owners April 15, 2025 15:37
@kapbh kapbh requested review from divipillai and umapraseeda April 15, 2025 15:37
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Apr 15, 2025
@kapbh kapbh added this to the 3.0.0 milestone Apr 15, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 15, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 1

Inputs:

Sources:

sdk-nrf: PR head: da8773c1be3ef6f4715266fd5f892239e7f96984

more details

sdk-nrf:

PR head: da8773c1be3ef6f4715266fd5f892239e7f96984
merge base: aac3ef5cf1627016b86f12ab4ceb751c1ce489c7
target head (main): aac3ef5cf1627016b86f12ab4ceb751c1ce489c7
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (3)
doc
│  ├── nrf
│  │  ├── includes
│  │  │  │ wifi_credentials_shell.txt
samples
│  ├── net
│  │  ├── http_server
│  │  │  │ README.rst
│  ├── wifi
│  │  ├── shell
│  │  │  │ README.rst

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

Copy link
Contributor

@krish2718 krish2718 left a comment

Choose a reason for hiding this comment

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

@kapbh
Copy link
Contributor Author

kapbh commented Apr 15, 2025

There is one reference in https://github.com/nrfconnect/sdk-zephyr/blob/main/subsys/net/lib/wifi_credentials/wifi_credentials_shell.c#L275

I will raise upstream PR and bring to NCS.

@krish2718
Copy link
Contributor

There is one reference in https://github.com/nrfconnect/sdk-zephyr/blob/main/subsys/net/lib/wifi_credentials/wifi_credentials_shell.c#L275

I will raise upstream PR and bring to NCS.

We can do that as part of nrfconnect/sdk-zephyr#2749, let's merge this separately.

@github-actions
Copy link

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-21834/nrf/samples/net/http_server/README.html
https://ncsdoc.z6.web.core.windows.net/PR-21834/nrf/samples/wifi/shell/README.html

Copy link
Contributor

@richabp richabp left a comment

Choose a reason for hiding this comment

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

Please add a changelog entry for this update as a suggestion in the #21339 PR.

@kapbh kapbh mentioned this pull request Apr 16, 2025
3 tasks
@nordicjm nordicjm merged commit 153c22b into nrfconnect:main Apr 16, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport v3.0-branch changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants