-
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
[CI] Add Matter[Pre/Post]CommandReceivedCallback #10500
[CI] Add Matter[Pre/Post]CommandReceivedCallback #10500
Conversation
PR #10500: Size comparison from 267951c to cd9f00a 34 builds
|
src/app/zap-templates/templates/app/im-cluster-command-handler.zapt
Outdated
Show resolved
Hide resolved
cd9f00a
to
36fda10
Compare
PR #10500: Size comparison from a56c83e to 36fda10 8 builds
14 builds
2 builds
10 builds
|
src/app/zap-templates/templates/app/im-cluster-command-handler.zapt
Outdated
Show resolved
Hide resolved
9c4d67d
to
36fda10
Compare
@mrjerryjohns I'm re-requesting review, not because I disagree with your comments, but because I think this is a more general tasks than adding back some features that were existing before IM takes over the command sending/handling. As said I would be happy to work on it as soon as possible (likely in a few weeks though :/ ) |
PR #10500: Size comparison from f0b6cbc to 36fda10 8 builds
14 builds
12 builds
|
36fda10
to
8ec6d8b
Compare
PR #10500: Size comparison from 4c7f664 to 8ec6d8b 2 builds (for p6, telink)
18 builds (for efr32, k32w, linux, qpg)
2 builds (for mbed)
12 builds (for esp32, nrfconnect)
|
Size increase report for "nrfconnect-example-build" from 4c7f664
Full report output
|
Size increase report for "esp32-example-build" from 4c7f664
Full report output
|
Fast tracking this, given we're past 3 days, and this is blocking testing. @vivien-apple can you resolve conflicts please? |
Fast tracking pending CI, given enough time for review has passed and comments seem resolved |
/rebase |
…is happening onto the stack
8ec6d8b
to
7033115
Compare
I have updated a bit the code so there is no stub generation anymore, but instead hardcoded stubs. This is not the final solution I would like but this will unlock CI so it is probably better to not wait for the final solution and land that as if. |
PR #10500: Size comparison from e24e76c to 7033115 Increases (29 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (37 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
…is happening onto the stack (project-chip#10500)
Problem
It uses to be a
emberAfPreCommandReceived
callback. This is useful in some contexts, notably in CI where you want to observer what is happening on the device.This PR adds it back as
Matter[Pre/Post]CommandReceivedCallback(const chip::app::ConcreteCommandPath & commandPath)
Change overview
Matter[Pre/Post]CommandReceivedCallback(const chip::app::ConcreteCommandPath & commandPath)
Testing
At the moment there is no consumer of this API but I'm planning to add some into some accessory test.