feat(report): add diff library to improve comparison feature#24
Conversation
|
that looks excellent, @dberna2! I'll review it today 💯 |
rieckpil
left a comment
rieckpil
left a comment
There was a problem hiding this comment.
Neat library, thanks for the hint 🚀
I've cloned your fork and used your branch to test it with ./renderReport.sh(in the root of the repo to quickly generate the report).
Somehow the comparison for the activeProfiles is off:
To replicate, use the mentioned script and compare context-0 with context-2 and check the difference in the profiles.
I'd assume the the difftool to only highlight the integration profile as the difference.
Can you check what's going wrong there?
|
I found the issue, I'm working on it and will update the PR as soon as I can 🙂 |
|
|
Hi @rieckpil! Note: |
|
@dberna2 excellent work! I've just tested it locally and really like it. Thanks a lot for this great feature update 👍 
|
2 participants
The default visual representation in the Detailed Comparison section is not very intuitive for visually identifying differences between contexts.
This change introduces the use of the diff library to provide a clearer visualization, highlighting which values have been added, removed, or modified. The goal is to make comparing the contents of both contexts simpler and more straightforward.
Current visualization
Proposed visualization