atspi icon indicating copy to clipboard operation
atspi copied to clipboard

Maintanance fixes

Open luukvanderduim opened this issue 1 year ago • 3 comments

Review with rigor, handle with care, scrutinize like it's 1999!

See individual commits for details.

luukvanderduim avatar May 15 '24 11:05 luukvanderduim

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.69%. Comparing base (85d7518) to head (f1696af). Report is 13 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #184      +/-   ##
==========================================
+ Coverage   83.74%   86.69%   +2.95%     
==========================================
  Files          39       39              
  Lines        3112     3338     +226     
==========================================
+ Hits         2606     2894     +288     
+ Misses        506      444      -62     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 15 '24 11:05 codecov[bot]

Thank you again Luuk for catching these important issues! I really would have never noticed, because I know the history and would have brushed it off.

I'm curious why you chose to upgrade zbus and zvarinat though? Generally, we want to use the oldest compatible version of dependencies. It would be a shame if we caused a version mismatch downstream just by being too aggressive about upstream upgrades.

TTWNO avatar May 16 '24 19:05 TTWNO

Z-crates Changelogs

zbus 4.2.1 has a number of "Critical fixes", especially the one where properties are invalidated after querying(?) seems problematic. Also "Hack to make connection handshake work inside Flatpak. xdg-dbus-proxy" might be of importance for us. zbus 4.2.0 dropped two dependencies.

I chose to also upgrade zvariant to keep the z-crates underlying dependencies in sync.

But if you want to investigate this a bit further, I am fine if you drop this commit and just take the others.

luukvanderduim avatar May 16 '24 21:05 luukvanderduim