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.

Sign up for GitHub

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

Jump to bottom

ref: exp show --only-changed #2966

Merged
merged 8 commits into from
Nov 4, 2021
Merged

ref: exp show --only-changed #2966

merged 8 commits into from
Nov 4, 2021

Conversation

daavoo
Copy link
Contributor

@daavoo daavoo commented Oct 25, 2021 •
edited
Loading

docs updates for iterative/dvc#6867

Not sure if this will actually get merged before #2862 but seems relevant to eventually include the option in the guide.

@daavoo daavoo added ⌛ status: wait-core-merge Waiting for related product PR merge/release C: ref Content of /doc/*-reference labels Oct 25, 2021
@daavoo daavoo self-assigned this Oct 25, 2021
@daavoo daavoo requested review from iesahin and dberenbaum October 25, 2021 20:16
@shcheklein shcheklein temporarily deployed to dvc-org-exp-show-only-c-bl7pmm October 25, 2021 20:17 Inactive
@daavoo daavoo mentioned this pull request Oct 25, 2021
Merged
2 tasks
@daavoo daavoo requested a review from sroy3 October 26, 2021 08:36
@dberenbaum
Copy link
Contributor

Pending resolution of comments in #6867

dberenbaum
dberenbaum approved these changes Oct 27, 2021
View reviewed changes
content/docs/command-reference/exp/show.md Outdated Show resolved Hide resolved
@restyled-io restyled-io bot mentioned this pull request Oct 27, 2021
Merged
@shcheklein shcheklein temporarily deployed to dvc-org-exp-show-only-c-bl7pmm October 27, 2021 12:56 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-exp-show-only-c-bl7pmm October 27, 2021 12:57 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-exp-show-only-c-bl7pmm October 27, 2021 17:36 Inactive
@shcheklein shcheklein temporarily deployed to dvc-org-exp-show-only-c-bl7pmm October 27, 2021 17:38 Inactive
jorgeorpinel
jorgeorpinel approved these changes Nov 3, 2021
View reviewed changes
Copy link
Contributor

@jorgeorpinel jorgeorpinel left a comment

Choose a reason for hiding this comment

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

Couple copy edits ☝️

@jorgeorpinel jorgeorpinel mentioned this pull request Nov 3, 2021
Closed
15 tasks
@jorgeorpinel
Copy link
Contributor

seems relevant to eventually include the option in the guide

Added to #2995

@restyled-io restyled-io bot mentioned this pull request Nov 3, 2021
Merged
@shcheklein shcheklein temporarily deployed to dvc-org-exp-show-only-c-bl7pmm November 3, 2021 08:34 Inactive
@jorgeorpinel jorgeorpinel merged commit b34f003 into master Nov 4, 2021
@jorgeorpinel jorgeorpinel deleted the exp-show-only-changed branch November 4, 2021 05:27
@jorgeorpinel
Copy link
Contributor

Oops I merged before it's even merged in the core repo... It's minor so I think we can let it be (doubt users will find it in docs before it's released)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dberenbaum dberenbaum dberenbaum approved these changes

@jorgeorpinel jorgeorpinel jorgeorpinel approved these changes

@iesahin iesahin Awaiting requested review from iesahin

@sroy3 sroy3 Awaiting requested review from sroy3

Assignees

@daavoo daavoo

Labels
C: ref Content of /doc/*-reference ⌛ status: wait-core-merge Waiting for related product PR merge/release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@daavoo @dberenbaum @jorgeorpinel @shcheklein