Skip to content

plots modify: complain if template does not exist#3995

Merged
skshetry merged 1 commit into
treeverse:masterfrom
skshetry:plots-modify-missing-template
Jun 10, 2020
Merged

plots modify: complain if template does not exist#3995
skshetry merged 1 commit into
treeverse:masterfrom
skshetry:plots-modify-missing-template

Conversation

@skshetry
Copy link
Copy Markdown
Collaborator

@skshetry skshetry commented Jun 9, 2020

  • ❗ I have followed the Contributing to DVC checklist.

  • 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here. If the CLI API is changed, I have updated tab completion scripts.

  • ❌ 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. 🙏

@skshetry skshetry self-assigned this Jun 9, 2020
Comment thread dvc/repo/plots/__init__.py Outdated
@skshetry skshetry requested review from Suor, efiop and pared and removed request for Suor June 9, 2020 14:18
Comment thread dvc/repo/plots/__init__.py Outdated
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

plots modify should not try to generate lockfile.

Comment thread dvc/repo/plots/__init__.py Outdated
Comment thread tests/func/plots/__init__.py Outdated
Copy link
Copy Markdown
Collaborator Author

@skshetry skshetry Jun 9, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to fix these ugly hacks: singele_stage being the default, later.

@efiop
Copy link
Copy Markdown
Contributor

efiop commented Jun 9, 2020

@skshetry Could you rebase, please?

@skshetry skshetry force-pushed the plots-modify-missing-template branch from c567f9e to 02d2e84 Compare June 10, 2020 02:20
@skshetry skshetry force-pushed the plots-modify-missing-template branch from 02d2e84 to 5c559be Compare June 10, 2020 02:23
Comment thread tests/func/plots/test_plots.py
Copy link
Copy Markdown
Contributor

@pared pared left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 comment

Comment thread tests/func/plots/test_plots.py
@skshetry skshetry merged commit 1d6c800 into treeverse:master Jun 10, 2020
@skshetry skshetry deleted the plots-modify-missing-template branch June 10, 2020 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants