Skip to content

problem_report: add CompressedFile.write() - #610

Open
bdrung wants to merge 2 commits into
canonical:mainfrom
bdrung:write-method
Open

problem_report: add CompressedFile.write()#610
bdrung wants to merge 2 commits into
canonical:mainfrom
bdrung:write-method

Conversation

@bdrung

@bdrung bdrung commented Apr 28, 2026

Copy link
Copy Markdown
Member

Add a CompressedFile.write() method to support this problem report value type in Report.add_kernel_crash_info().

@codecov

codecov Bot commented Apr 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.49%. Comparing base (2eacb11) to head (6ecefca).

Files with missing lines Patch % Lines
problem_report.py 90.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #610   +/-   ##
=======================================
  Coverage   84.48%   84.49%           
=======================================
  Files         106      106           
  Lines       21018    21029   +11     
  Branches     3195     3196    +1     
=======================================
+ Hits        17758    17769   +11     
  Misses       2785     2785           
  Partials      475      475           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bdrung
bdrung force-pushed the write-method branch 2 times, most recently from dc6dcc2 to 570655b Compare July 2, 2026 20:05
@bdrung
bdrung requested a review from Hyask July 3, 2026 15:10
bdrung added 2 commits August 4, 2026 01:22
The method `decode_compressed_stream` is used outside of
`CompressedValue`. Move this method to a stand-alone function to ease
reusing it.
Add a `CompressedFile.write()` method to support this problem report
value type in `Report.add_kernel_crash_info()`. This change is part of
the wider approach to add type hints to `ProblemReport`.
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.

1 participant