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 revision information to ID docs #37539

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cecille
Copy link
Contributor

@cecille cecille commented Feb 12, 2025

Draft right now, new doc needs to be added to the doc index files.

Testing

Documentation generation - md files look fine in preview.

Copy link

semanticdiff-com bot commented Feb 12, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  scripts/spec_xml/generate_spec_xml.py  31% smaller
  scripts/spec_xml/paths.py  9% smaller
  docs/ids_and_codes/spec_clusters.md Unsupported file format
  docs/ids_and_codes/spec_device_types.md Unsupported file format

@github-actions github-actions bot added documentation Improvements or additions to documentation scripts labels Feb 12, 2025
|95 |0x005F |Microwave Oven Control |Y |Y |Y |Y |
|96 |0x0060 |Operational State |Y |Y |Y |Y |
|97 |0x0061 |RVC Operational State |Y |Y |Y |Y |
|98 |0x0062 |Scenes Management |Y |P |P |P |
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm aware this is auto generated, but the Y in 1.3 is wrong, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah - will need to re-pull the data model files at 1.3.0.1. Spec tag at 1.3 still has this as non-provisional

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm going to pause this one and do a re-pull of 1.3 before posting this. Thanks, Rene.

@@ -0,0 +1,101 @@
# List of currently defined spec clusters
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the title was originally wrong. It should be corrected:

Suggested change
# List of currently defined spec clusters
# List of currently defined device types

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants