Skip to content

Fix/diff report - #559

Merged
erikbosch merged 2 commits into
COVESA:masterfrom
jdacoello:fix/diff-report
Sep 8, 2026
Merged

erikbosch merged 2 commits into
COVESA:masterfrom
jdacoello:fix/diff-report

Conversation

@jdacoello

Copy link
Copy Markdown
Contributor

Enhanced the diff report expected by ddmodl ledger tool to be more consistent.

Signed-off-by: JD Alvarez <8550265+jdacoello@users.noreply.github.com>
…iff reporting

Signed-off-by: JD Alvarez <8550265+jdacoello@users.noreply.github.com>
@erikbosch

Copy link
Copy Markdown
Collaborator

Candidtae to also be include din 6.1 release

@sschleemilch - would be great if you could take a quick look at the python code.

@erikbosch erikbosch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, but leave it open for some time to give others a chance to review. If merged, I will cherry pick it to 6.x branch

@erikbosch
erikbosch merged commit 97c5e34 into COVESA:master Sep 8, 2026
5 checks passed
erikbosch pushed a commit that referenced this pull request Sep 8, 2026
* fix: update ddmodl version to 0.3.0 and enhance diff reporting

Signed-off-by: JD Alvarez <8550265+jdacoello@users.noreply.github.com>

* fix: update ddmodl version to 0.4.0 and enhance is_leaf handling in diff reporting

Signed-off-by: JD Alvarez <8550265+jdacoello@users.noreply.github.com>

---------

Signed-off-by: JD Alvarez <8550265+jdacoello@users.noreply.github.com>
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.

2 participants