-
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
[chip-tool] Move WebSocketServer code to src/app/tests/suites/server … #24617
[chip-tool] Move WebSocketServer code to src/app/tests/suites/server … #24617
Conversation
80f4899
to
828f546
Compare
PR #24617: Size comparison from 9de347f to 828f546 Increases (6 builds for bl702, esp32, linux, telink)
Decreases (8 builds for bl602, bl702, cyw30739, nrfconnect, psoc6, telink)
Full report (54 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
src/app is taken as part of Matter core SDK in coverage statistic, if WebSocketServer is only reused by example apps, why not move to some place under examples, such as examples/common? |
|
Currently, WebSocketServer is under 'examples/chip-tool/commands/interactive', so I think this is not used only for the testing. The idea is we want to set a clear boundary between the example code and core SDK code, we conducts daily coverage statistic for Matter Core SDK for our internal reference, we don't count the code coverage in the examples now, so it is better to keep all example code outside of core SDK directories. |
…so it can be reused
828f546
to
49e348f
Compare
I have moved the component in |
PR #24617: Size comparison from 6e12e92 to 49e348f Increases (10 builds for bl602, bl702, cc13x2_26x2, linux, nrfconnect, psoc6, telink)
Decreases (6 builds for cc13x2_26x2, esp32, telink)
Full report (43 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
…so it can be reused (project-chip#24617)
…so it can be reused
Problem
This PR just moves
chip-tool
web socket server code to a place where other apps can used it.