Skip to content

Conversation

@skshetry
Copy link
Collaborator

@skshetry skshetry commented May 6, 2020

Note: This does not change default for pipeline as
it cannot show multiple DAGs at the same time. If we
are able to show multiple DAGs, we can just default to
dvc.yaml.

treeverse/dvc.org#1173

  • ❌ I will check DeepSource, CodeClimate, and other sanity checks below. (We consider them recommendatory and don't expect everything to be addressed. Please fix things that actually improve code or fix bugs.)

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

Note: This does not change default for `pipeline` as
it cannot show multiple DAGs at the same time. If we
are able to show multiple DAGs, we can just default to
`dvc.yaml`.
@skshetry skshetry requested review from efiop, pared and pmrowla May 6, 2020 11:44
@skshetry skshetry self-assigned this May 6, 2020
@efiop efiop merged commit e383106 into treeverse:master May 6, 2020
@skshetry skshetry deleted the repro-use-dvcyaml branch May 6, 2020 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhances DVC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants