-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
add CLI tab-to-autocomplete ability and implement for nrfconnect #13630
Conversation
PR #13630: Size comparison from 9109dff to a9b5af9 Increases above 0.2%:
Increases (6 builds for esp32, linux, mbed, nrfconnect, telink)
Decreases (6 builds for esp32, linux, mbed, nrfconnect, telink)
Full report (32 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
a9b5af9
to
69a9ee8
Compare
PR #13630: Size comparison from d611c63 to 69a9ee8 Increases above 0.2%:
Increases (5 builds for linux, mbed, nrfconnect, telink)
Decreases (5 builds for linux, mbed, nrfconnect, telink)
Full report (30 builds for efr32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
dcbe1d4
to
b7295ce
Compare
PR #13630: Size comparison from b6e08a8 to b7295ce Increases above 0.2%:
Increases (6 builds for cyw30739, linux, mbed, nrfconnect, telink)
Decreases (6 builds for cyw30739, linux, mbed, nrfconnect, telink)
Full report (31 builds for cyw30739, efr32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
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.
Fast tracking, given > 3 days of review & tooling changes.
PR #13630: Size comparison from 246473c to e09593f Increases (2 builds for cyw30739, telink)
Decreases (3 builds for cyw30739, linux, telink)
Full report (15 builds for cyw30739, efr32, k32w, linux, p6, qpg, telink)
|
/rebase |
e09593f
to
1846966
Compare
PR #13630: Size comparison from 812f820 to 1846966 Increases (4 builds for cyw30739, esp32, nrfconnect, telink)
Decreases (5 builds for cyw30739, esp32, linux, nrfconnect, telink)
Full report (32 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
…ject-chip#13630) * implement cli autocompletion for ESP32 and nRFConnect * avoid STL;revert implementation for esp32 * update to register with nullptr prefix * cleanup comment * revert an unnecessary change * backfill two new commands; address CI failures
…ect (project-chip#13630)" (project-chip#14384) This reverts commit c62489b.
Problem
Change overview
Engine::GetCmdCompletion
to provide command completion based on command prefixesTesting
Build and tested manually with
nRF52840dk and nRF52840dongle with lighting-app example

updated Jan 26: I couldn't get the esp32 implementation to work properly after addressing review comments, so removing the esp32 implementation from this PR and will do another PR later once fixed.