Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dvc: rename plot to plots #3802

Merged
merged 2 commits into from May 14, 2020
Merged

dvc: rename plot to plots #3802

merged 2 commits into from May 14, 2020

Conversation

efiop
Copy link
Member

@efiop efiop commented May 14, 2020

Follows dvc params/metrics convention and is needed in preparation for new
commands and some refactoring.

Part of #3409

iterative/dvc.org#1293

  • ❌ 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. πŸ™

@efiop efiop requested a review from pared May 14, 2020 17:07
efiop added a commit to iterative/dvc.org that referenced this pull request May 14, 2020
@efiop
Copy link
Member Author

efiop commented May 14, 2020

Ok, should also rename .dvc/plot -> .dvc/plots. We have a lot of files in there that are describing different plots.

EDIT: done

Follows `dvc params/metrics` convention and is needed in preparation for new
commands and some refactoring.

Part of iterative#3409
efiop added a commit to iterative/dvc.org that referenced this pull request May 14, 2020
@efiop efiop changed the title dvc: rename plot to plots [WIP] dvc: rename plot to plots May 14, 2020
@efiop efiop changed the title [WIP] dvc: rename plot to plots dvc: rename plot to plots May 14, 2020
@efiop efiop merged commit cec4b4e into iterative:master May 14, 2020
@efiop efiop deleted the 3409 branch May 14, 2020 19:51
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.

None yet

2 participants