Skip to content

Bug/update gen templates json file for nested files/zapp 1297#1577

Merged
brdandu merged 1 commit into
project-chip:masterfrom
brdandu:bug/updateGenTemplatesJsonFileForNestedFiles/ZAPP-1297
Apr 23, 2025
Merged

Bug/update gen templates json file for nested files/zapp 1297#1577
brdandu merged 1 commit into
project-chip:masterfrom
brdandu:bug/updateGenTemplatesJsonFileForNestedFiles/ZAPP-1297

Conversation

@brdandu
Copy link
Copy Markdown
Collaborator

@brdandu brdandu commented Apr 18, 2025

When json files referenced from within the templates.json file are updated then checksum is calculated for those as well and updated accordingly.

@brdandu
Copy link
Copy Markdown
Collaborator Author

brdandu commented Apr 18, 2025

Tests already covered in zcl-loader.test.js test changing the zcl extension file in a top level templates json file and make sure it is re-loaded again

@brdandu brdandu force-pushed the bug/updateGenTemplatesJsonFileForNestedFiles/ZAPP-1297 branch 2 times, most recently from 807af61 to 24e77ba Compare April 22, 2025 18:16
…on files within it are updated. The newly added templates json file is in sync and the older ones will be out of sync in the package table when a nested json file is changed

- Removing the zcl extensions check because that is now covered by the newly added code. Not adding further tests for this because it is already covered in zcl-loader.test.js. See: test changing the zcl extension file in a top level templates json file and make sure it is re-loaded again
- JIRA: ZAPP-1297
@brdandu brdandu force-pushed the bug/updateGenTemplatesJsonFileForNestedFiles/ZAPP-1297 branch from 24e77ba to 38a2fe3 Compare April 23, 2025 12:49
@brdandu brdandu merged commit ff46f97 into project-chip:master Apr 23, 2025
13 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.

3 participants