Skip to content

--stdin and --all for dvc check-ignore cmd#1675

Merged
jorgeorpinel merged 17 commits into
treeverse:masterfrom
karajan1001:fix_1674
Aug 19, 2020
Merged

--stdin and --all for dvc check-ignore cmd#1675
jorgeorpinel merged 17 commits into
treeverse:masterfrom
karajan1001:fix_1674

Conversation

@karajan1001
Copy link
Copy Markdown
Contributor

@karajan1001 karajan1001 commented Aug 7, 2020

Fixes #1674 (to match treeverse/dvc/pull/4323).

Only a draft one.

You may disregard these recommendations if you used the Edit on GitHub button from dvc.org to improve a doc in place.

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.

🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.

Please choose to allow us to edit your branch when creating the PR.

Thank you for the contribution - we'll try to review it as soon as possible. 🙏

@jorgeorpinel
Copy link
Copy Markdown
Contributor

Only a draft one.

Are you aware of the draft PR functionality in Github @karajan1001 ? 🙂

https://github.blog/2019-02-14-introducing-draft-pull-requests/

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

I changed a few things. Just a couple things left 🙂

p.s. this will probably have conflicts with #1673 though...

Comment thread content/docs/command-reference/check-ignore.md Outdated
@jorgeorpinel
Copy link
Copy Markdown
Contributor

jorgeorpinel commented Aug 8, 2020

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

Just one thing left ☝️ Thanks

@jorgeorpinel
Copy link
Copy Markdown
Contributor

Looks like I'll have to take this one over.

@jorgeorpinel jorgeorpinel merged commit 63fd0ba into treeverse:master Aug 19, 2020
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.

doc: arguments --stdin and --all for dvc check-ignore cmd

3 participants