vscode-dvc icon indicating copy to clipboard operation
vscode-dvc copied to clipboard

Refine data status consumption

Open mattseddon opened this issue 3 years ago • 1 comments

2/2 main <- #2091 <- this

This PR improves our use of the new data status command.

We now:

  • Show resources in multiple SCM groups when they are returned by data status.
  • Apply a single decoration to each resource based on a prioritized list. We apply not in cache first, then uncommitted, then committed statuses the implementation matches the git extension's approach.
  • Do not show checkout inline for resources that are uncommitted and also not in cache (action would not work).

Demo

https://user-images.githubusercontent.com/37993418/183329636-d9b3e429-0a92-4c61-8cab-95e66c2d63ea.mov

Note: Still need https://github.com/iterative/dvc/issues/8062 to be fixed before I can provide a non-confusing demo of not in cache behaviour.

mattseddon avatar Aug 06 '22 05:08 mattseddon

Code Climate has analyzed commit e85a1671 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 2

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 96.7% (0.0% change).

View more on Code Climate.

qlty-cloud-legacy[bot] avatar Aug 31 '22 23:08 qlty-cloud-legacy[bot]