-
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
chef sample app #16731
Merged
andy31415
merged 14 commits into
project-chip:master
from
rochaferraz:experimental/rochaferraz/github-patch1-master
Apr 6, 2022
Merged
chef sample app #16731
andy31415
merged 14 commits into
project-chip:master
from
rochaferraz:experimental/rochaferraz/github-patch1-master
Apr 6, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…during the build process Change-Id: Ia6aeba2d5587e09074defcd34dc5028288d74a90
Change-Id: I33f572f313dc7be91e208b074fe6ba6c7ab9da31
pullapprove
bot
requested review from
anush-apple,
Byungjoo-Lee,
bzbarsky-apple,
carol-apple,
cecille,
chrisdecenzo,
chshu,
chulspro,
Damian-Nordic,
dhrishi,
electrocucaracha,
emargolis,
franck-apple,
gjc13,
harimau-qirex,
hawk248,
harsha-rajendran,
isiu-apple,
jelderton,
jepenven-silabs,
jmartinez-silabs,
kghost,
kpschoedel,
lazarkov,
LuDuda,
mlepage-google and
msandstedt
March 28, 2022 17:58
dnicoara
reviewed
Mar 30, 2022
boring-cyborg
bot
added
config
documentation
Improvements or additions to documentation
github
workflows
labels
Mar 31, 2022
Change-Id: If7c287ea7e9632fb8b4a49e8e4c82af4002cf4e4
rochaferraz
force-pushed
the
experimental/rochaferraz/github-patch1-master
branch
from
March 31, 2022 20:18
033488c
to
863a8ec
Compare
dnicoara
reviewed
Mar 31, 2022
PR #16731: Size comparison from e8c3452 to 863a8ec Increases (1 build for esp32)
Full report (39 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
PR #16731: Size comparison from e8c3452 to 91b40bd Increases (1 build for esp32)
Full report (39 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
woody-apple
approved these changes
Apr 1, 2022
Fast tracking testing changes. @rochaferraz Can you please re-generate zap? |
Should it be merged into shell/ ? |
mspang
reviewed
Apr 5, 2022
Change-Id: I8a353828804b2a203d336e47edbe58ce7d62163a
rochaferraz
force-pushed
the
experimental/rochaferraz/github-patch1-master
branch
from
April 5, 2022 17:28
91b40bd
to
19a5fc1
Compare
Change-Id: I252594388ec759c84525ea9090c153bdd7f80d3f
andy31415
reviewed
Apr 5, 2022
PR #16731: Size comparison from bc01144 to 42ca0d2 Increases (1 build for esp32)
Full report (36 builds for cc13x2_26x2, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
Change-Id: Ia17baae1a7cca206fd1854928f1b56ca05c0b9e7
PR #16731: Size comparison from bc01144 to 2682d93 Increases above 0.2%:
Increases (12 builds for esp32, linux)
Decreases (34 builds for cc13x2_26x2, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Full report (36 builds for cc13x2_26x2, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
andy31415
approved these changes
Apr 6, 2022
PR #16731: Size comparison from 964b4c2 to 2922409 Increases (1 build for esp32)
Full report (25 builds for cc13x2_26x2, efr32, esp32, k32w, mbed, nrfconnect, p6, telink)
|
andrei-menzopol
pushed a commit
to andrei-menzopol/connectedhomeip
that referenced
this pull request
Apr 14, 2022
* new: chef app is a configurable application that processes zap files during the build process Change-Id: Ia6aeba2d5587e09074defcd34dc5028288d74a90 * fix: esp32 functional on master on 28/mar/2021 Change-Id: I33f572f313dc7be91e208b074fe6ba6c7ab9da31 * chg: removed old zap files that need to be updated Change-Id: Ibbe26ad39ba69d251fd4ef956632eb1aac9dfdf8 * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Restyled by prettier-markdown * Restyled by autopep8 * fix: init server on scheduled task Change-Id: If7c287ea7e9632fb8b4a49e8e4c82af4002cf4e4 * fix: excluded unsued context from task call Change-Id: I8a353828804b2a203d336e47edbe58ce7d62163a * chg: fixes to paths in README file Change-Id: I252594388ec759c84525ea9090c153bdd7f80d3f * fix: update to zap regen script to ignore chef sample app Change-Id: Ia17baae1a7cca206fd1854928f1b56ca05c0b9e7 * Update wordlist: add new work (flags gzbf) and sort|unique the word list Co-authored-by: Restyled.io <[email protected]> Co-authored-by: Andrei Litvin <[email protected]>
vinitg1
pushed a commit
to vinitg1/connectedhomeip
that referenced
this pull request
Apr 22, 2022
* new: chef app is a configurable application that processes zap files during the build process Change-Id: Ia6aeba2d5587e09074defcd34dc5028288d74a90 * fix: esp32 functional on master on 28/mar/2021 Change-Id: I33f572f313dc7be91e208b074fe6ba6c7ab9da31 * chg: removed old zap files that need to be updated Change-Id: Ibbe26ad39ba69d251fd4ef956632eb1aac9dfdf8 * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Restyled by prettier-markdown * Restyled by autopep8 * fix: init server on scheduled task Change-Id: If7c287ea7e9632fb8b4a49e8e4c82af4002cf4e4 * fix: excluded unsued context from task call Change-Id: I8a353828804b2a203d336e47edbe58ce7d62163a * chg: fixes to paths in README file Change-Id: I252594388ec759c84525ea9090c153bdd7f80d3f * fix: update to zap regen script to ignore chef sample app Change-Id: Ia17baae1a7cca206fd1854928f1b56ca05c0b9e7 * Update wordlist: add new work (flags gzbf) and sort|unique the word list Co-authored-by: Restyled.io <[email protected]> Co-authored-by: Andrei Litvin <[email protected]>
andy31415
added a commit
that referenced
this pull request
Apr 22, 2022
* chef sample app (#16731) * new: chef app is a configurable application that processes zap files during the build process Change-Id: Ia6aeba2d5587e09074defcd34dc5028288d74a90 * fix: esp32 functional on master on 28/mar/2021 Change-Id: I33f572f313dc7be91e208b074fe6ba6c7ab9da31 * chg: removed old zap files that need to be updated Change-Id: Ibbe26ad39ba69d251fd4ef956632eb1aac9dfdf8 * Restyled by whitespace * Restyled by clang-format * Restyled by gn * Restyled by prettier-markdown * Restyled by autopep8 * fix: init server on scheduled task Change-Id: If7c287ea7e9632fb8b4a49e8e4c82af4002cf4e4 * fix: excluded unsued context from task call Change-Id: I8a353828804b2a203d336e47edbe58ce7d62163a * chg: fixes to paths in README file Change-Id: I252594388ec759c84525ea9090c153bdd7f80d3f * fix: update to zap regen script to ignore chef sample app Change-Id: Ia17baae1a7cca206fd1854928f1b56ca05c0b9e7 * Update wordlist: add new work (flags gzbf) and sort|unique the word list Co-authored-by: Restyled.io <[email protected]> Co-authored-by: Andrei Litvin <[email protected]> * Chef Sample App - yaml configuration file (#17330) * chg: removed config file, SDK path needs to be set as environment variable Change-Id: Ied6a186f5d7370818461f921e76309a0508aaaf5 * chg: config file is now a YAML file Change-Id: Ia34b45c1e54245b94da06dca626170a78581b7b9 * chg: zap-generated files in independent folders by device type Change-Id: If86a393576166a03f1aa801947b36233308e6df4 * fix: fixes CMakeLists files for ESP32 and nrfconnect to build on current tot Change-Id: Ia6f94dfc6a186036f41af1c567b4b8416ff2ef22 * chg: applied restyle Change-Id: I38f52568e0a7f967b35ad42ca7ade1373235c3fe * fix: added a words to .wordslist Change-Id: I84a7cb6ea7830146711450d0d713b9da2fd895b4 * Update .wordlist.txt * Update README.md * Update README.md * Update .wordlist.txt * fix: reverted unwanted change to submodule hash reference Change-Id: I15fee54e792d8aa06d83051b803c9e27093d29d3 * Restyled by autopep8 Co-authored-by: Douglas Rocha Ferraz <[email protected]> Co-authored-by: Restyled.io <[email protected]> Co-authored-by: Andrei Litvin <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
config
documentation
Improvements or additions to documentation
examples
fast track
github
review - pending
workflows
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The purpose of the chef app is to to increase the coverage of device types in
Matter.
It uses the shell app a starting point, adding the processing of ZAP files and
the support of a few targets under a unified build script:
chef.py
.As it incorporates the processing of ZAP files as part of the build process, it
does not use
zzz_generated
, but rather places the auto-generated zapartifacts under its
zap-generated
temporary folder.All device types available (DM/IM .zap files) are found inside the
devices
folder.
Change overview
New sample App under examples/chef
#16639 is similar and targets interop_testing_2022_01_18 branch