[pull] master from project-chip:master #628
Merged
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
* Allow to communicate with DUT via standard input * Use fabric-sync-app.py stdin instead dedicated pipe * Drop pipe stdin forwarder in fabric-sync-app.py * Restyled by autopep8 * Wait for thread to stop * Fix referencing not-created variable --------- Co-authored-by: Restyled.io <commits@restyled.io>
* Implement Android DiagnosticLog * Update from comment * Restyled * Update DiagnosticLog UX (Download complete / failed) * Fix timeout issue * Fix java-controller build error * modify from comment * Add javadoc, test cases * kotlin code style * Fix from comment * Fix build error
- Changed the initial value of max to 1 within the function. - If max is not updated during the for loop, it returns -1, which can cause an INTEGER_OVERFLOW - By initializing max to 1, it ensures that if max is updated during the loop, the updated value is used. - If no value is updated, it performs the -1 operation and updates to 0.
* Python testing: Fix some type annotation errors mypy no longer shows these errors. To be clear, it shows lots of OTHER errors, but these are some low-hanging fruit that are fairly easily cleaned up. * s/typing.Optional/Optional/g
… it is not needed (#36722)
* [Python] Pass DAC verifier on SetupCommissioner Besides setting the global DAC verifier, also explicitly pass the verifier on controller initialization. This avoids the common Controller initialization code to complain with: ``` *** Missing DeviceAttestationVerifier configuration at DeviceCommissioner init: using global default, consider passing one in CommissionerInitParams. ``` * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>
* ESP32: Fix IEEE802154 MAC address in diagnostic provider * Restyled by clang-format --------- Co-authored-by: Restyled.io <commits@restyled.io>
* [Telink] Update Docker image (Zephyr update) * [Telink] Update Docker image (Zephyr update)
* Disable flaky test * Fix disabling logic * make disabling logic consistent in java
…ch are not caused by WriteAttribute. (#36364) * data-model-provider: Add an interface to report attribute changes which are not caused by WriteAttribute * review changes * Restyled by clang-format * Update src/app/data-model-provider/Provider.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * Update src/app/data-model-provider/Provider.h Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> * review changes * remove the deprecated mark to fix CI issue * review changes * Restyled by clang-format * fix java build * move the report change callback to metadatatree * fix CI build --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Boris Zbarsky <bzbarsky@apple.com> Co-authored-by: Andrei Litvin <andy314@gmail.com>
…) and ember-specific (placed in codegen level) (#36658) * Separate out logic between Safe and Ember storage persistence. - keeps persistence and safe persistence as interfaces - implementations are now Separate - let server initialize the default safe persistence - update codegen provider to initialize the ember side * Some manual updates * Scripted update * Restyle * Some manual updates again * More manual updates. Only tests remain (and these are odd now ....) * Update tests and support for nullptr, manual fix for one compilation * Remove ember dependency from server * Fix more typos for compilation * Restyle * Add a note about cadmin 1_19 being slow * Fix up link to PR for persistence * Restyled by prettier-markdown * Mark one more slow test * Add a few more dependencies to make esp32 link * fix data model paths * Fix qpg and nrf compile * Fix a few more tests * Restyled by clang-format * Fix another typo for NRF compile * Add persistence cpp files to darwin project * Fix typo * Drop timeouts for java tests: it is not reasonable to wait 360 minutes for a run to timeout, set individual tests to short * Add back deleted line from before * Restyled by clang-format * Update src/app/DefaultSafeAttributePersistenceProvider.h Co-authored-by: Andy Salisbury <salisbury.andy@gmail.com> --------- Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: Andy Salisbury <salisbury.andy@gmail.com>
* Check fabric-sync application with CCTRL tests on CI * Add support to local.py * Setup stdin pipe to keep app running
* Build fabric-sync for testing * Test TC_ECOINFO test with new fabric-sync-app on CI * Restyled by isort --------- Co-authored-by: Restyled.io <commits@restyled.io>
…use matter sdk specific APIs but native APIS (#36723)
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.1)
Can you help keep this open source service alive? 💖 Please sponsor : )