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

guide: add more links to the .dvcignore guide from other docs #1033

Closed
1 of 8 tasks
jorgeorpinel opened this issue Mar 4, 2020 · 7 comments
Closed
1 of 8 tasks

guide: add more links to the .dvcignore guide from other docs #1033

jorgeorpinel opened this issue Mar 4, 2020 · 7 comments
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: guide Content of /doc/user-guide good first issue Good for newcomers help wanted Contributors especially welcome type: enhancement Something is not clear, small updates, improvement suggestions

Comments

@jorgeorpinel
Copy link
Contributor

jorgeorpinel commented Mar 4, 2020

To help increase awareness about this feature!

Per iterative/dvc#3257 (comment)

UPDATE: Here's a non-inclusive list of some ideas of the most obvious places where mentions of .dvcignore should exist. Each one could be addressed in different PRs:

BTW if while attempting any of the above bugs or strange behavior arise, please ask in Discord or open an issue in the core repo (iterative/dvc)

@jorgeorpinel jorgeorpinel added type: enhancement Something is not clear, small updates, improvement suggestions good first issue Good for newcomers A: docs Area: user documentation (gatsby-theme-iterative) labels Mar 4, 2020
@sarthakforwet
Copy link
Contributor

Hey @jorgeorpinel . As far as I got this all you mean to say is that we have to refer the https://dvc.org/doc/user-guide/dvcignore from other sections of DVC docs?

@jorgeorpinel
Copy link
Contributor Author

Yes @sarthakforwet! You would first need to play with and fully understand the .dvcignore feature, then explore all the existing docs and try to find all the places where it would be appropriate to link to the .dvcignore guide.

But I see you're also interested in #1172 (comment), let's focus on a single one for now 🙂 please pick one and feel free to submit a PR to close it, thanks.

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented May 23, 2020

@sarthakforwet are you still interested in this issue or #1172? Both involve .dvcignore which I'm guessing you've studied already so at this point focusing on these is probably the best bet. Thanks

@jorgeorpinel

This comment has been minimized.

@jorgeorpinel

This comment has been minimized.

@jorgeorpinel jorgeorpinel added the good first issue Good for newcomers label May 23, 2020
@sarthakforwet
Copy link
Contributor

@jorgeorpinel, Yes, I am interested in both but unable to get #1172 correctly about what we have to do . Do we need to play with all external dependencies i.e SSH , Amazon S3 , Google Cloud Storage etc.. and check for them with respect to .dvcignore ?

@jorgeorpinel
Copy link
Contributor Author

Do we need to play with all external dependencies

No, you could just play with the most basic one i.e. an external directory path in the same file system.

But you're already doing PR #1346 so let's focus on that for now 🙂

@jorgeorpinel jorgeorpinel changed the title user-guide: add more links to the .dvcignore guide from other docs guide: add more links to the .dvcignore guide from other docs Sep 26, 2021
@iesahin iesahin added the C: guide Content of /doc/user-guide label Oct 21, 2021
@dberenbaum dberenbaum closed this as not planned Won't fix, can't repro, duplicate, stale Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative) C: guide Content of /doc/user-guide good first issue Good for newcomers help wanted Contributors especially welcome type: enhancement Something is not clear, small updates, improvement suggestions
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants