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

Add fingerprinters and profiles for certification #889

Closed
wants to merge 0 commits into from
Closed

Add fingerprinters and profiles for certification #889

wants to merge 0 commits into from

Conversation

chenpeixiang
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commented Aug 3, 2023

CLA assistant check
All committers have signed the CLA.

@lelandblue lelandblue added needs invite link Needs the bestow invite like do not merge labels Aug 18, 2023
@github-actions
Copy link

Duplicate profile check: Passed - no duplicate profiles detected.

@github-actions
Copy link

github-actions bot commented Aug 18, 2023

Test Results

     52 files     336 suites   0s ⏱️
1 577 tests 1 577 ✔️ 0 💤 0
2 732 runs  2 732 ✔️ 0 💤 0

Results for commit d442c9d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 18, 2023

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

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against d442c9d

@@ -0,0 +1,22 @@
name: light-color-level-1800-6500K
Copy link
Contributor

@ctowns ctowns Aug 21, 2023

Choose a reason for hiding this comment

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

This should be light-color-level-1800K-6500K (notice the extra K after 1800) to match naming scheme of other similar fingerprints. Please change the name: and the filename, then update the deviceProfileName in the fingerprints to match light-color-level-1800K-6500K

@lelandblue
Copy link
Contributor

Hello @chenpeixiang I just want to confirm, are you still interested in persuing this device for WWST Certification? We noticed a change on your application. If this was not intentional please reach out to [email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge needs invite link Needs the bestow invite like
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants