Simplify flag handling in qvm_ls#447
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #447 +/- ##
==========================================
+ Coverage 76.24% 76.36% +0.11%
==========================================
Files 53 53
Lines 9322 9282 -40
==========================================
- Hits 7108 7088 -20
+ Misses 2214 2194 -20 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026041101-4.3&flavor=pull-requests Test run included the following:
New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026032404-devel&flavor=update
Failed tests18 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/170766#dependencies 31 fixed
Unstable testsDetails
Performance TestsPerformance degradation:9 performance degradations
Remaining performance tests:102 tests
|
Purpose
qvm_lshas a huge chunk of code dedicated to handling flags - that appears to be remnant of an old, mostly unused implementation.This PR greatly simplifies this logic by transforming
flagsinto just another standardColumn.This PR shouldn't change any user-facing feature.
Minor changes
Columns