-
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
RVC - add tests to CI #28665
RVC - add tests to CI #28665
Conversation
PR #28665: Size comparison from 9a35c44 to d3fe55c Increases (11 builds for bl602, cyw30739, esp32, psoc6, telink)
Decreases (7 builds for bl702, efr32, telink)
Full report (60 builds for bl602, bl702, bl702l, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, 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.
This needs a codegen rerun....
PR #28665: Size comparison from 9a35c44 to ac0f955 Increases (1 build for cc32xx)
Full report (1 build for cc32xx)
|
PR #28665: Size comparison from 3c3e5f5 to 778a881 Increases (1 build for cc32xx)
Full report (1 build for cc32xx)
|
This reverts commit 778a881.
PR #28665: Size comparison from 3c3e5f5 to ad2aafb Increases (4 builds for bl602, cc32xx, nrfconnect)
Decreases (1 build for bl702l)
Full report (15 builds for bl602, bl702, bl702l, cc32xx, k32w, mbed, nrfconnect, qpg)
|
PR #28665: Size comparison from 3c3e5f5 to 06beb49 Increases (9 builds for bl602, bl702l, cc32xx, nrfconnect, telink)
Decreases (15 builds for bl702, efr32, esp32, nrfconnect, psoc6, telink)
Full report (60 builds for bl602, bl702, bl702l, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
@cecille @andy31415 QQ since Boris is on vacation - do we need a zap regen when adding tests to CI? It doesn't look like it, but please let me know if that's actually needed. |
* add changes * add missing PIXIT arg * add codegen changes * Revert "add codegen changes" This reverts commit 778a881. * add missing space
You used to for chip-tool until #28783 merged, no? And you still do for darwin-framework-tool. But this PR is not really adding the test to CI, since in |
project-chip#28665 didn't enable the YAML bits.
project-chip#28665 didn't enable the YAML bits.
project-chip#28665 didn't enable the YAML bits.
#28665 didn't enable the YAML bits.
* add changes * add missing PIXIT arg * add codegen changes * Revert "add codegen changes" This reverts commit 778a881. * add missing space
project-chip#28665 didn't enable the YAML bits.
No description provided.