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

repro: implement downstream option #1926

Merged
merged 3 commits into from Apr 25, 2019
Merged

repro: implement downstream option #1926

merged 3 commits into from Apr 25, 2019

Conversation

ghost
Copy link

@ghost ghost commented Apr 24, 2019

  • Have you followed the guidelines in our Contributing document?

  • Does your PR affect documented changes or does it add new functionality that should be documented? If yes, have you created a PR for dvc.org documenting it or at least opened an issue for it? If so, please add a link to it.

iterative/dvc.org#271


Close #1555

@ghost ghost requested a review from efiop April 24, 2019 19:28
@efiop
Copy link
Member

efiop commented Apr 24, 2019

@MrOutis Just a heads up: your python3x tests failed.

@ghost
Copy link
Author

ghost commented Apr 24, 2019

Thanks, @efiop , didn't notice it before 👍
The PR is ready to be reviewed, by the way :)

efiop
efiop previously requested changes Apr 24, 2019
tests/func/test_repro.py Outdated Show resolved Hide resolved
@ghost ghost dismissed efiop’s stale review April 24, 2019 23:02

implemented your suggestions, @efiop, do you mind reviewing it again?

@ghost ghost requested a review from efiop April 24, 2019 23:05
tests/func/test_repro.py Outdated Show resolved Hide resolved
@ghost ghost requested a review from efiop April 25, 2019 00:11
tests/unit/test_repro.py Outdated Show resolved Hide resolved
efiop
efiop previously requested changes Apr 25, 2019
tests/unit/test_repro.py Outdated Show resolved Hide resolved
@ghost ghost requested a review from efiop April 25, 2019 01:31
@ghost ghost dismissed efiop’s stale review April 25, 2019 01:31

Moved the unit test under command and test both with and without downstream

scripts/completion/dvc.zsh Outdated Show resolved Hide resolved
Co-Authored-By: mroutis <mroutis@protonmail.com>
dvc/command/repro.py Outdated Show resolved Hide resolved
Co-Authored-By: mroutis <mroutis@protonmail.com>
Copy link
Member

@efiop efiop left a comment

Choose a reason for hiding this comment

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

👍

@efiop efiop merged commit c0104e2 into iterative:master Apr 25, 2019
@ghost ghost deleted the fix-1555 branch December 13, 2019 23:31
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.

repro: continue downstream
1 participant