-
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 - yaml configuration file #17330
Merged
andy31415
merged 12 commits into
project-chip:master
from
rochaferraz:experimental/rochaferraz/github-patch1-master-configless-rebased
Apr 20, 2022
Merged
Chef Sample App - yaml configuration file #17330
andy31415
merged 12 commits into
project-chip:master
from
rochaferraz:experimental/rochaferraz/github-patch1-master-configless-rebased
Apr 20, 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
Contributor
rochaferraz
commented
Apr 13, 2022
- Changed chef sample app configuration file to yaml. It picks up a defaults from environment variables first time the script is run
- Builds on current tot for esp32, nrfconnect and linux targets
- nrf52840dk_nrf52840 renamed to nrfconnect
…iable Change-Id: Ied6a186f5d7370818461f921e76309a0508aaaf5
Change-Id: Ia34b45c1e54245b94da06dca626170a78581b7b9
Change-Id: If86a393576166a03f1aa801947b36233308e6df4
…ent tot Change-Id: Ia6f94dfc6a186036f41af1c567b4b8416ff2ef22
andy31415
approved these changes
Apr 13, 2022
PR #17330: Size comparison from a75cd99 to 0b958a0 Full report (16 builds for cc13x2_26x2, cyw30739, k32w, linux, mbed, p6, telink)
|
pullapprove
bot
requested review from
anush-apple,
Byungjoo-Lee,
bzbarsky-apple,
carol-apple,
chrisdecenzo,
chshu,
chulspro,
Damian-Nordic,
dhrishi,
electrocucaracha,
franck-apple,
gjc13,
hawk248,
harsha-rajendran,
isiu-apple,
jelderton,
jepenven-silabs,
jmartinez-silabs,
lazarkov,
LuDuda,
msandstedt,
sagar-apple and
saurabhst
April 13, 2022 14:49
pullapprove
bot
requested review from
selissia,
tecimovic,
vijs,
vivien-apple,
wbschiller,
woody-apple and
xylophone21
April 13, 2022 14:49
Change-Id: I38f52568e0a7f967b35ad42ca7ade1373235c3fe
Change-Id: I84a7cb6ea7830146711450d0d713b9da2fd895b4
PR #17330: Size comparison from a75cd99 to a137e3f Increases above 0.2%:
Increases (39 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Full report (39 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
…ter-configless-rebased
PR #17330: Size comparison from b63d897 to e685a2f Full report (39 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
bzbarsky-apple
approved these changes
Apr 13, 2022
andy31415
reviewed
Apr 19, 2022
Change-Id: I15fee54e792d8aa06d83051b803c9e27093d29d3
vinitg1
pushed a commit
to vinitg1/connectedhomeip
that referenced
this pull request
Apr 22, 2022
* 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
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
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.