exp: add --hide-workspace flag to exp show

[MirjalilovS]

Fixes #10741

[MirjalilovS]

Hi, I am failing a lot of CI checks and I am not sure how I can fix that. I followed the DVC Contributing Docs and managed to pass every test on my local machine. I have updated the relevant tests to account for the new feature. Any help would be greatly appreciated.

Also, should documentation be created for this? I noticed whilst working on this issue that there are some exp show flags are not mentioned in the documentation such as --hide-queue and --hide-failed. This left me quite confused as to how I should proceed. Again, any help would be greatly appreciated.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.05%. Comparing base (2431ec6) to head (8e215bc).
⚠️ Report is 98 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10798      +/-   ##
==========================================
+ Coverage   90.68%   91.05%   +0.37%     
==========================================
  Files         504      504              
  Lines       39795    40376     +581     
  Branches     3141     3195      +54     
==========================================
+ Hits        36087    36766     +679     
+ Misses       3042     2978      -64     
+ Partials      666      632      -34     

☔ View full report in Codecov by Sentry.
📢 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.

[skshetry]

Hi, I am failing a lot of CI checks and I am not sure how I can fix that. I followed the DVC Contributing Docs and managed to pass every test on my local machine. I have updated the relevant tests to account for the new feature. Any help would be greatly appreciated.

This was due to an upstream bug: jelmer/dulwich#1643, which is now been resolved.

Also, should documentation be created for this? I noticed whilst working on this issue that there are some exp show flags are not mentioned in the documentation.

Great finding. yes, this and all of those flags need to be documented. Looks like we forgot to update the docs when we introduced those flags in #8318.

[MirjalilovS]

Thank you for your response. When making an issue for this in the docs repo, should it be separate for each feature or is it fine to put it all in one issue?

[skshetry]

Thank you for your response. When making an issue for this in the docs repo, should it be separate for each feature or is it fine to put it all in one issue?

It might be easier to create a PR directly than creating an issue. If you don't want to document other flags right now, it's okay to create just a single issue to track them.

[skshetry]

@MirjalilovS, would you be able to add a test for this?

[MirjalilovS]

@MirjalilovS, would you be able to add a test for this?

Hi, I added a test for the flag in tests/unit/command/test_experiments.py. I could not find any other tests for exp show flags so I assumed that was all that I had to do. Please let me know if there is anything else that needs to be added.