Skip to content
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

Production Edge Driver PR for 29-July-2024 #1551

Merged
merged 13 commits into from
Jul 29, 2024
Merged

Production Edge Driver PR for 29-July-2024 #1551

merged 13 commits into from
Jul 29, 2024

Conversation

lelandblue
Copy link
Contributor

Description of Change

This is the PR to start the production release for 29-July-2024.

seojune79 and others added 13 commits July 15, 2024 16:28
…es (#1494)

* change the way windowShadeLevel capability updates

* change open and close command handlers

* modified windowShadeLevel update process and added new test case
* Edge Driver Repo Check List

This is a potential sample for a PR Template for the Edge Drivers Repo.

* Update Checklist

* Update Checklist

* Update Checklist

* Rename Checklist to PULL_REQUEST_TEMPLATE

Change file name to PULL_REQUEST_TEMPLATE

* Update PULL_REQUEST_TEMPLATE

* Update PULL_REQUEST_TEMPLATE

* Update PULL_REQUEST_TEMPLATE

* Update and rename PULL_REQUEST_TEMPLATE to .github/PULL_REQUEST_TEMPLATE.md

Relocate and change file type

* Update PULL_REQUEST_TEMPLATE.md

* Update PULL_REQUEST_TEMPLATE.md
* Remove embedded clusters from CI coverage results
Co-authored-by: Harrison Carter <[email protected]>
* new-device-AiDot-Orein- OS0100411267

* Adding additional Fingerprint for Linkind device

* Adding additional fingerprint for the Oriein OS01006110
* new-devices-for-Cync-and-Winees

* Adding additional Fingerprint for 

Additional PR to add another recently opened WWST CERT request for a Cync Full Color A19, ModelNumber: CLEDA199CD1MS.  

vendorId: 0x1339
productId: 0x0015

I could not confirm the Matter Device Type ID

* Fix a bad commit

Removed a dupe fingerprint and added the following fingerprint. 

- id: 4921/107
  deviceLabel: Cync Reveal Full Color A19
  vendorId: 0x1339
  productId: 0x006B
  deviceProfileName: light-color-level-2000K-7000K

* Adding missed quotes around product ID
Matter Switch: add missing quotes around fingerprint id
Revert (Zigbee Window Treatment) Aqara/Roller Shade PR 1494
@lelandblue lelandblue requested review from ctowns, greens and cjswedes July 29, 2024 19:06
Copy link

Duplicate profile check: Passed - no duplicate profiles detected.

Copy link

github-actions bot commented Jul 29, 2024

Channel deleted.

Copy link

Test Results

   60 files    377 suites   0s ⏱️
1 827 tests 1 827 ✅ 0 💤 0 ❌
3 176 runs  3 176 ✅ 0 💤 0 ❌

Results for commit e48ae8b.

Copy link

matter-switch_coverage.xml

File Coverage
All files 94%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/init.lua 96%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/eve-energy/init.lua 91%

matter-thermostat_coverage.xml

File Coverage
All files 80%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/init.lua 96%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-switch/src/eve-energy/init.lua 91%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-thermostat/src/init.lua 83%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/matter-thermostat/src/embedded-cluster-utils.lua 42%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against e48ae8b

Copy link
Contributor

@ctowns ctowns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, but I'm not sure why the Delete PR channel check is being shown here. It looks like that CI check was run 4 hours ago for #1544 but for some reason it is showing up here. It looks like the PR channel for #1544 is still open so I think we'd just need to run the delete channel task again and it should be resolved. @greens have we seen the delete channel task show up on a different PR before?

@greens
Copy link
Contributor

greens commented Jul 29, 2024

@ctowns nope! Not sure why that happened but I suspect github weirdness.

@lelandblue lelandblue merged commit 436cd1b into production Jul 29, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants