docs: clarifications around external outputs info. #2154
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Merged
This comment has been minimized.
This comment has been minimized.
jorgeorpinel
commented
Mar 12, 2021
jorgeorpinel
commented
Mar 12, 2021
| # Managing External Data | ||
| # External Outputs |
There was a problem hiding this comment.
Apparently this name was quite intentional as "outputs" is not meaningful for end-users (at the user guide level)...
|
I reverted the guide name change in 49eefb0 @shcheklein. PTAL when you have a chance. |
shcheklein
approved these changes
Mar 14, 2021
shcheklein
pushed a commit
that referenced
this pull request
Mar 29, 2021
) (#2214) * guide: disclaim x data (impro #2104) * Added changes from PR #2188 and modified paths & titles - Changes title of "Data Access" to "Data and Model Access" - Changes title of "Data Versioning" to "Data and Model Versioning" - Renames path of Data Access and Data Versioning to `data-and-model-access.md` and `data-and-model-versioning.md` respectively. - Adds redirects -- `/doc/start/data-access` -> `/doc/start/data-and-model-access` -- `/doc/start/data-versioning` -> `/doc/start/data-and-model-versioning` - Replaces links in `/doc/start` with the new links. * Update redirects-list.json with fixed subsection redirects. Co-authored-by: Jorge Orpinel <[email protected]> * Fixed incomplete looking sentence * merged into a single paragraph * Divided models sentence and added "large files" phrase. * Adds new paths to sidebar * Updated links to data-access and data-versioning cmd ref * updated links to data-access and data-versioning in blog * Updated links to data-access and data-versioning in UC * Updated links to data-access and data-versioning in UG * updated yarn.lock * Update content/docs/start/data-and-model-versioning.md Co-authored-by: Jorge Orpinel <[email protected]> * Restyled by prettier * fixes hardcoded links to data-and-model-access in the blog * minor fixes * guide: revert Exp Outs guide rename per #2154 (review) * start: emphasize models are files (assumption) * start: roll back unnecessary changes unnecessary for #2214 Co-authored-by: Jorge Orpinel <[email protected]> Co-authored-by: Jorge Orpinel <[email protected]> Co-authored-by: Emre Sahin <iex@levinas> Co-authored-by: Restyled.io <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rename "Managing External Data" guide to "External Outputs"add/import-url --to-cache/remote