-
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
[YAML] Update scripts/tools/zap_regen_all.py to be able to generate only the test targets if needed #15246
[YAML] Update scripts/tools/zap_regen_all.py to be able to generate only the test targets if needed #15246
Conversation
PR #15246: Size comparison from d671a52 to eebe6a1 Full report (43 builds for cyw30739, efr32, esp32, 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.
Approving, but this showcases how it takes a significant amount of deep maintenance knowledge of ZAP and associated helpers to be able to do simple optimizations of workflow. So many hidden and unexplained conventions building on top of each other...
eebe6a1
to
638598b
Compare
638598b
to
9d6f08a
Compare
PR #15246: Size comparison from 90e379e to 9d6f08a Full report (43 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
9d6f08a
to
bbf20e3
Compare
PR #15246: Size comparison from e71be1c to bbf20e3 Full report (34 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
bbf20e3
to
f0fd8d2
Compare
PR #15246: Size comparison from 7b4b1a8 to f0fd8d2 Full report (34 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Sigh. |
… test targets if needed
f0fd8d2
to
46a789d
Compare
PR #15246: Size comparison from 9039ba0 to 46a789d Full report (34 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
When one wants to generate the tests content from YAML, it needs to generates much more than what is needed.
Change overview
scripts/tools/zap_regen_all.py
to make it possible to generate only tests / a specific test targetTesting
I have locally run
scripts/tools/zap_regen_all.py
with and without some options