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

Bug : Summary diff number differs from HTML report in some cases (not really determined) #171

Open
LasneF opened this issue Nov 26, 2024 · 1 comment

Comments

@LasneF
Copy link
Contributor

LasneF commented Nov 26, 2024

given the same command line

HTML report gives this
image

vs summary report gives this

❌ 1 Breaking changes out of 46
INFO Modifications: 5
INFO Removals: 24
INFO Additions: 14

looks the 46 a bit weird as summing the info it gives 43 , as same as on the HTML report

@LasneF
Copy link
Contributor Author

LasneF commented Nov 26, 2024

further investigation show case that it may be linked toward handling adding a path in the new file

here is the summary

Date: 11/26/24 | Commit: New: .\perforce copy.yaml, Original: component-descriptor.yml
Document Element | Total Changes | Breaking Changes
paths | 34 | 1
components | 14 | 0

❌ 1 Breaking changes out of 48
INFO Modifications: 9
INFO Removals: 24
INFO Additions: 12

but here is the report json

{
"reportSummary": {
"components": {
"totalChanges": 14,
"breakingChanges": 0
},
"paths": {
"totalChanges": 34,
"breakingChanges": 1
}
},

and the HTML report
image

something may be to handle at same time as this
#148

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

No branches or pull requests

1 participant