Merged
Conversation
Signed-off-by: James Ball <jameball@qti.qualcomm.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR extends the params-generation pipeline to support a new csr_definitions section in param definition YAML, producing a csrs collection in the generated params JSON/HTML output and updating schemas/tests accordingly.
Changes:
- Add CSR ingestion (
csr_definitions) intools/create_params.py, emittingcsrsentries and rendering CSR tables/links in HTML output. - Update feature rendering fallback to prefix chapter-based features with
CHAP:for clearer display. - Extend JSON schemas to describe
csr_definitionsinputs andcsrsoutputs; update golden test fixtures and a Makefile loop.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/shared_utils.py | Changes feature formatting fallback to CHAP:<chapter> when no impl-def-derived features exist. |
| tools/create_params.py | Adds CSR parsing/output, HTML sidebar/table rendering for CSRs, and generalizes impl-def error messaging. |
| tests/params/test-ch2.yaml | Adds csr_definitions test input. |
| tests/params/expected/test-params.json | Updates expected params JSON to include csrs. |
| tests/params/expected/test-params.html | Updates expected HTML to include CSR sidebar section and CSR table(s). |
| tests/params/expected/test-param-table-variants/**/ROSE_COLORS.adoc | Updates expected AsciiDoc fragments due to CHAP: feature formatting. |
| tests/params/expected/test-param-appendix-adoc-includes/test-ch2/ROSE_COLORS.adoc | Updates expected AsciiDoc include output due to CHAP: feature formatting. |
| tests/norm-rule/test-ch2.yaml | Adjusts normative rule fixtures to support CSR field rules and alters ROSE_COLORS fixture. |
| tests/norm-rule/expected/test-norm-rules.json | Updates expected normative-rules JSON output to reflect fixture changes. |
| tests/norm-rule/expected/test-norm-rules.html | Updates expected normative-rules HTML output to reflect fixture changes. |
| schemas/params-schema.json | Adds csrs output schema and loosens top-level required keys via anyOf. |
| schemas/param-defs-schema.json | Adds csr_definitions input schema and loosens required keys via anyOf. |
| schemas/param-common-schema.json | Adds csrType enum used by the new CSR output schema. |
| Makefile | Replaces a shell for loop with a $(foreach ...) expansion for param-table variants generation. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: James Ball <jameball@qti.qualcomm.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.