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

Improve artifact comparison delta results #3300

Merged

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Dec 22, 2023

Currently only the text is written as detailed information into the artifactcomparision directory.

This is often not enough information to find the actual difference, to mitigate this the handling of the writing is now delegated to the delta and the following improvements are made:

  • class file comparison writes the class file as well as the disassemble
  • zip comparator writes the baseline and build jars

@laeubi laeubi added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label Dec 22, 2023
@laeubi laeubi force-pushed the Improve_artifact_comparison_delta_results branch from 7626205 to 0fe3631 Compare December 22, 2023 15:11
Currently only the text is written as detailed information into the
artifactcomparision directory.

This is often not enough information to find the actual difference, to
mitigate this the handling of the writing is now delegated to the delta
and the following improvements are made:

- class file comparison writes the class file as well as the disassemble
- zip comparator writes the baseline and build jars
@laeubi laeubi force-pushed the Improve_artifact_comparison_delta_results branch from 0fe3631 to 7299732 Compare December 22, 2023 15:13
Copy link

Test Results

   576 files  ±0     576 suites  ±0   3h 48m 29s ⏱️ + 2m 26s
   380 tests ±0     374 ✔️ ±0    6 💤 ±0  0 ±0 
1 140 runs  ±0  1 121 ✔️ ±0  19 💤 ±0  0 ±0 

Results for commit 7299732. ± Comparison against base commit 3cfab28.

@laeubi laeubi merged commit b3910ef into eclipse-tycho:main Dec 22, 2023
10 checks passed
Copy link

💚 All backports created successfully

Status Branch Result
tycho-4.0.x

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant