Skip to content

exp remove: add --rev and -n options#3193

Merged
jorgeorpinel merged 4 commits into
treeverse:masterfrom
karajan1001:add_rev_to_exp_remove
Apr 11, 2022
Merged

exp remove: add --rev and -n options#3193
jorgeorpinel merged 4 commits into
treeverse:masterfrom
karajan1001:add_rev_to_exp_remove

Conversation

@karajan1001
Copy link
Copy Markdown
Contributor

@karajan1001 karajan1001 commented Jan 18, 2022

@iesahin iesahin added C: ref Content of /doc/*-reference ⌛ status: wait-core-merge Waiting for related product PR merge/release labels Jan 18, 2022
@karajan1001 karajan1001 force-pushed the add_rev_to_exp_remove branch from 48085a8 to 663a7f5 Compare January 24, 2022 08:27
@jorgeorpinel jorgeorpinel temporarily deployed to dvc-org-add-rev-to-exp--aorvo1 January 27, 2022 04:23 Inactive
@jorgeorpinel

This comment has been minimized.

Copy link
Copy Markdown
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.

Never mind on re-opening the PR, I'm reviewing this as-is... Let's just try to remember for future PRs 🙂 Thanks @karajan1001

Comment thread content/docs/command-reference/exp/remove.md Outdated
Comment thread content/docs/command-reference/exp/remove.md Outdated
@jorgeorpinel jorgeorpinel changed the title exp remove: add new flags --rev and -n exp remove: add --rev and -n options Jan 27, 2022
Comment thread content/docs/command-reference/exp/show.md Outdated
@jorgeorpinel

This comment has been minimized.

@shcheklein shcheklein temporarily deployed to dvc-org-add-rev-to-exp--aorvo1 January 27, 2022 13:20 Inactive
@karajan1001 karajan1001 force-pushed the add_rev_to_exp_remove branch from e5f7acb to e4181d1 Compare January 27, 2022 13:45
@shcheklein shcheklein temporarily deployed to dvc-org-add-rev-to-exp--aorvo1 January 27, 2022 13:45 Inactive
@karajan1001 karajan1001 force-pushed the add_rev_to_exp_remove branch from e4181d1 to 51ecccd Compare January 27, 2022 13:46
@shcheklein shcheklein temporarily deployed to dvc-org-add-rev-to-exp--aorvo1 January 27, 2022 13:46 Inactive
@karajan1001 karajan1001 force-pushed the add_rev_to_exp_remove branch from 51ecccd to ba32a0a Compare February 16, 2022 07:32
@shcheklein shcheklein temporarily deployed to dvc-org-add-rev-to-exp--aorvo1 February 16, 2022 07:33 Inactive
@karajan1001 karajan1001 force-pushed the add_rev_to_exp_remove branch from ba32a0a to 2604377 Compare February 16, 2022 07:37
@shcheklein shcheklein temporarily deployed to dvc-org-add-rev-to-exp--aorvo1 February 16, 2022 07:37 Inactive
Comment thread content/docs/command-reference/exp/remove.md Outdated
@shcheklein shcheklein temporarily deployed to dvc-org-add-rev-to-exp--aorvo1 March 8, 2022 12:32 Inactive
Comment thread content/docs/command-reference/exp/remove.md Outdated
@shcheklein shcheklein temporarily deployed to dvc-org-add-rev-to-exp--aorvo1 March 9, 2022 07:31 Inactive
@jorgeorpinel jorgeorpinel removed the C: ref Content of /doc/*-reference label Mar 9, 2022
@dberenbaum dberenbaum removed the ⌛ status: wait-core-merge Waiting for related product PR merge/release label Mar 9, 2022
@shcheklein shcheklein had a problem deploying to dvc-org-add-rev-to-exp--aorvo1 March 15, 2022 09:37 Failure
@julieg18

This comment was marked as resolved.

karajan1001 and others added 3 commits March 15, 2022 19:46
Co-authored-by: Jorge Orpinel <jorgeorpinel@users.noreply.github.com>
@karajan1001 karajan1001 force-pushed the add_rev_to_exp_remove branch from c6b937b to 31176e6 Compare March 15, 2022 11:46
@shcheklein shcheklein temporarily deployed to dvc-org-add-rev-to-exp--aorvo1 March 15, 2022 11:47 Inactive
@jorgeorpinel
Copy link
Copy Markdown
Contributor

jorgeorpinel commented Mar 23, 2022

Has the feature been released or is close to? If so please merge this or lmk and I will. Thanks

@dberenbaum
Copy link
Copy Markdown
Contributor

It will be released in 2.9.6, which will likely happen in the next week or so.

@jorgeorpinel jorgeorpinel added the ⌛ status: wait-core-merge Waiting for related product PR merge/release label Mar 30, 2022
@jorgeorpinel
Copy link
Copy Markdown
Contributor

Well we merged all the others already so... 🚀

@jorgeorpinel jorgeorpinel merged commit 5b15f3d into treeverse:master Apr 11, 2022
@karajan1001 karajan1001 deleted the add_rev_to_exp_remove branch April 12, 2022 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⌛ status: wait-core-merge Waiting for related product PR merge/release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants