Skip to content

Commit 66ac089

Browse files
authored
Merge pull request #84 from AryaHassanli/fix-zap-device-render
Fix zap device type rendering
2 parents d914ad5 + ed1a68d commit 66ac089

2 files changed

Lines changed: 2 additions & 5 deletions

File tree

zap/render/composed.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -202,8 +202,8 @@ func getConformanceState(cxt conformance.Context, clusterRequirements []*matter.
202202
}
203203

204204
switch req.Origin {
205-
case matter.RequirementOriginBaseDeviceType, matter.RequirementOriginSubsetDeviceType:
206-
// Normally, we do not include clusters from the Base Device Type or subset device types...
205+
case matter.RequirementOriginSubsetDeviceType:
206+
// Normally, we do not include clusters from subset device types...
207207

208208
var hasElements bool
209209
ers := elementRequirements[req.ClusterRequirement.Cluster]

zap/render/conformance.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,6 @@ import (
1111

1212
func renderConformance(spec *spec.Specification, entity types.Entity, c conformance.Conformance, parent *etree.Element, alternatives ...string) error {
1313
removeConformance(parent)
14-
if conformance.IsMandatory(c) && !conformance.IsProvisional(c) {
15-
return nil
16-
}
1714
conformanceElement, err := dm.CreateConformanceElement(c, nil)
1815
if err != nil {
1916
return err

0 commit comments

Comments
 (0)