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

test/apport-valgrind: do not limit diff size #253

Merged
merged 1 commit into from Dec 7, 2023

Conversation

bdrung
Copy link
Collaborator

@bdrung bdrung commented Nov 16, 2023

Set maxDiff to None to see full diff output for better debugging bugs like https://launchpad.net/bugs/2043713

Set `maxDiff` to `None` to see full diff output for better debugging
bugs like https://launchpad.net/bugs/2043713

Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com>
Copy link

codecov bot commented Nov 16, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2fdb988) 83.80% compared to head (226ef03) 83.80%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #253   +/-   ##
=======================================
  Coverage   83.80%   83.80%           
=======================================
  Files          93       93           
  Lines       18837    18838    +1     
=======================================
+ Hits        15787    15788    +1     
  Misses       3050     3050           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@schopin-pro schopin-pro left a comment

Choose a reason for hiding this comment

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

Fair enough.

@schopin-pro schopin-pro merged commit 5ba9e06 into canonical:main Dec 7, 2023
25 checks passed
@bdrung bdrung deleted the full-log branch December 7, 2023 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants