Skip to content

repro 1.x : Updated downstream example and added info for Sequential execution.#1624

Merged
jorgeorpinel merged 18 commits into
treeverse:jorgefrom
sarthakforwet:repro_misc
Jul 31, 2020
Merged

repro 1.x : Updated downstream example and added info for Sequential execution.#1624
jorgeorpinel merged 18 commits into
treeverse:jorgefrom
sarthakforwet:repro_misc

Conversation

@sarthakforwet
Copy link
Copy Markdown
Contributor

PR as an outcome of discussion at #1572(review).

Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
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.

Correct intentions, thanks! Some details to fix below:

Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
@jorgeorpinel jorgeorpinel mentioned this pull request Jul 24, 2020
@jorgeorpinel
Copy link
Copy Markdown
Contributor

Also, the changes per #1572 (review) may also be missing still, but if it's too confusing or doesn't seem to apply now please lmk @sarthakforwet and I'll take care of it.

Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
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.

A few more changes needed ☝️

Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
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.

Want to take a quick final look before I merge this @shcheklein ? Thanks

And thanks again, @sarthakforwet

@shcheklein shcheklein temporarily deployed to dvc-landing-repro-misc-z88dnez July 31, 2020 20:33 Inactive
Comment thread content/docs/command-reference/repro.md Outdated
Comment thread content/docs/command-reference/repro.md Outdated
Copy link
Copy Markdown
Contributor

@shcheklein shcheklein left a comment

Choose a reason for hiding this comment

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

looks good to me! just a few typos.

@jorgeorpinel jorgeorpinel temporarily deployed to dvc-landing-repro-misc-z88dnez July 31, 2020 21:59 Inactive
@jorgeorpinel jorgeorpinel merged commit 012b72f into treeverse:jorge Jul 31, 2020
@jorgeorpinel
Copy link
Copy Markdown
Contributor

Merged #1648!

@sarthakforwet sarthakforwet deleted the repro_misc branch August 1, 2020 03:26
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