Fix #10638 : makes remove return correct list when used with both --queue and -A

[rmic]

Description

This PR addresses issue #10638, by providing a minimal code fix that makes the --all-commits(aka. -A flag) append to the existing removed list instead of replacing it with a new value.

When --queue is used, it might already contain the names of experiments removed from the queue. In all other cases, the list should exist but be empty, so appending still makes the job correctly.

Checklist

• ❗ I have followed the Contributing to DVC checklist.

• [N/A] 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.

Thank you for the contribution - we'll try to review it as soon as possible. 🙏

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.02%. Comparing base (2431ec6) to head (c2414b8).
Report is 11 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10641      +/-   ##
==========================================
+ Coverage   90.68%   91.02%   +0.34%     
==========================================
  Files         504      504              
  Lines       39795    39858      +63     
  Branches     3141     3152      +11     
==========================================
+ Hits        36087    36281     +194     
+ Misses       3042     2948      -94     
+ Partials      666      629      -37     

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

[shcheklein]

thanks, @rmic

[rmic]

thanks, @rmic

You're welcome ! Thanks for your guidance and patience 🙂