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: update links #4017

Merged
merged 5 commits into from
Oct 6, 2022
Merged

Conversation

jorgeorpinel
Copy link
Contributor

Follow-up to #4011 (review)

@shcheklein shcheklein temporarily deployed to dvc-org-guide-reorg-lin-pvgbxq October 5, 2022 03:50 Inactive
@jorgeorpinel jorgeorpinel changed the base branch from main to guide/reorg/links October 5, 2022 03:51
@jorgeorpinel jorgeorpinel mentioned this pull request Oct 5, 2022
2 tasks
@jorgeorpinel jorgeorpinel added A: docs Area: user documentation (gatsby-theme-iterative) C: guide Content of /doc/user-guide labels Oct 5, 2022
@jorgeorpinel
Copy link
Contributor Author

This is basically a multi-file find/replace + auto-formatting. I did a visual check of the many changes and they seem right. If the link checker passes, this should be mergeable.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2022

0525bc1

Link Check Report

All 80 links passed!

CML watermark

jorgeorpinel and others added 2 commits October 5, 2022 00:52
* start: remove Data Mgmt index pg and
move some of its contents to the GS index page

* Update content/docs/start/index.md
Co-authored-by: efiop <[email protected]>
Comment on lines -24 to +16
| `Workspace directory` | Filesystem type (e.g. ext4, FAT, etc.) and drive on which the <abbr>workspace</abbr> is mounted |
| `Repo` | Shows whether we are in a DVC repo and/or Git repo |
| Line | Detail |
| ------------------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These crazy long changes are due to linting: MD tables need to accommodate for the length of the updated links in every row...

@jorgeorpinel
Copy link
Contributor Author

6/80 links failed.

Fixed links ☝🏼

Waiting for updated check...

@shcheklein shcheklein temporarily deployed to dvc-org-guide-reorg-lin-pvgbxq October 6, 2022 21:59 Inactive
@jorgeorpinel
Copy link
Contributor Author

All links are passing now. Merging!

@jorgeorpinel jorgeorpinel merged commit bf0fef2 into guide/reorg/links Oct 6, 2022
@jorgeorpinel jorgeorpinel deleted the guide/reorg/links-4-real branch October 6, 2022 22:22
jorgeorpinel added a commit that referenced this pull request Oct 14, 2022
* guide: reorg sidebar

* guide: update sidebar plus redirects

* Update redirects-list.json

* guide: minor fixes to sidebar edits

* plots: more links for pcp
rel #4011 (review)

* fix admon typoe

* guide: update links (#4017)

* guide: update links based on
https://github.com/iterative/dvc.org/pull/4011/files#diff-a6828b19e9794c1fbb6799ea95263fb7487b676d054932b5f6e3b315eb12a6a2

* start: remove Data Mgmt index page (#4016)

* start: remove Data Mgmt index pg and
move some of its contents to the GS index page

* Update content/docs/start/index.md

* dvc 2.29.0 (#4021)

Co-authored-by: efiop <[email protected]>

* how-to: fix run-dvc-on-win links

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: efiop <[email protected]>

* Update src/components/DownloadButton/index.tsx

Co-authored-by: dberenbaum <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: efiop <[email protected]>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants