Skip to content

Migrating docs from source to docs repo - #6899

Merged
Aditya Patwardhan (adityapatwardhan) merged 3 commits into
PowerShell:masterfrom
sdwheeler:sdw-docs2441
Jun 1, 2018
Merged

Migrating docs from source to docs repo#6899
Aditya Patwardhan (adityapatwardhan) merged 3 commits into
PowerShell:masterfrom
sdwheeler:sdw-docs2441

Conversation

@sdwheeler

Copy link
Copy Markdown
Collaborator

PR Summary

The docs folder in this repo contains a lot of documentation about the PowerShell source code and build environment.
It also has contained documentation about about installing and using Powershell.
That documentation really belongs in the PowerShell/PowerShell-Docs repo.

We are in the process of migrating the user-focused article to the docs repo.
This file records which files have been migrated.

2018-05-18

The following files have been moved to the PowerShell/PowerShell-Docs repo.

Original file location New location in PowerShell/PowerShell-Docs
docs/installation/linux.md reference/docs-conceptual/setup/Installing-PowerShell-Core-on-Linux.md
docs/installation/macos.md reference/docs-conceptual/setup/Installing-PowerShell-Core-on-macOS.md
docs/installation/windows.md reference/docs-conceptual/setup/Installing-PowerShell-Core-on-Windows.md
docs/BREAKINGCHANGES.md reference/docs-conceptual/whats-new/breaking-changes-ps6.md
docs/KNOWNISSUES.md reference/docs-conceptual/whats-new/known-issues-ps6.md

PR Checklist

@iSazonov

Copy link
Copy Markdown
Collaborator

With removing the files we'll have many broken links internally and externally. Can we replace each removed file with a placeholder file with a description and link to new file?

@sdwheeler

Copy link
Copy Markdown
Collaborator Author

Ilya (@iSazonov) We should not be linking to docs on GitHub. We should only be linking to published docs on docs.microsoft.com. Can you provide some examples to help me understand the impact?

Also, I added a DOCSMIGRATION.md file with that information, it that not sufficient?

@iSazonov

Copy link
Copy Markdown
Collaborator

Sean Wheeler (@sdwheeler) I meant that issues in the repo contain links on the docs. Also somebody can have links in blog posts.

@sdwheeler

Copy link
Copy Markdown
Collaborator Author

Ilya (@iSazonov) That is another reason to not have docs like that in the source repo. In docs.microsoft.com we have a way to redirect those external links when they change. We cannot do that when the link points to GitHub.

@iSazonov

Copy link
Copy Markdown
Collaborator

Sean Wheeler (@sdwheeler) Please see the comment #6002 (comment)
If somebody press link there he get "GitHub page not found". I'd prefer to get a page with text and link on new document place "The document was moved. See new link".

@SteveL-MSFT

Copy link
Copy Markdown
Member

Ilya (@iSazonov), Joey Aiello (@joeyaiello) and I discussed this and in general linking directly to GitHub repo is not recommended. Linking directly to a commit guarantees that link will work, but won't get the latest revision. This is one of the reasons to move the docs not repo specific out so that deep links should continue to work.

@iSazonov

Copy link
Copy Markdown
Collaborator

Steve Lee (@SteveL-MSFT) I assume that most of these references to documentation do not use commits. People will do this in the future too and GitHub does not block it. Actually the problem is not this. People will get the Page not found, and we can show them a page with a link to the new document location. I suppose it will be more user friendly. I don't insist if you don't see the problem.

@SteveL-MSFT

Copy link
Copy Markdown
Member

Ilya (@iSazonov) seems like maybe we can use this to redirect https://help.github.com/articles/redirects-on-github-pages/ ?

@sdwheeler

Copy link
Copy Markdown
Collaborator Author

Steve Lee (@SteveL-MSFT) Ilya (@iSazonov) that redirection only applies to GitHub Pages published using Jekyll.

@SteveL-MSFT

Copy link
Copy Markdown
Member

Sean Wheeler (@sdwheeler) too bad. For now, I would suggest moving ahead as-is. Ilya (@iSazonov) you have a valid concern and we should be more aware of that going forward if we add new docs to this repo. If there is sufficient community feedback, we can consider putting replacement .md files that tell the user where the new documentation is.

@iSazonov

Copy link
Copy Markdown
Collaborator

LGTM.

@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit 1f918bb into PowerShell:master Jun 1, 2018
@bergmeister

Christoph Bergmeister (bergmeister) commented Jun 2, 2018

Copy link
Copy Markdown
Contributor

Not even the links of the main readme were updated to point to the docs site, this is not really a good migration process

@SteveL-MSFT

Copy link
Copy Markdown
Member

Sean Wheeler (@sdwheeler) can you submit a PR to update the links on the README.md?

@sdwheeler

Copy link
Copy Markdown
Collaborator Author

Steve Lee (@SteveL-MSFT) Will do. I had a previous PR for this but it got closed. Will work on a new one.

@bergmeister

Christoph Bergmeister (bergmeister) commented Jun 4, 2018

Copy link
Copy Markdown
Contributor

Sean Wheeler (@sdwheeler) see pr #6981 and feel free to modify it to your liking

@rkeithhill

Copy link
Copy Markdown
Collaborator

Folks, people rely on these installation instructions. I have internal documentation that points to these links so folks know how to install PS Core. It is way not cool to have these install links broken for multiple days. :-(

@sdwheeler
Sean Wheeler (sdwheeler) deleted the sdw-docs2441 branch April 25, 2019 14:23
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj-contribs/PowerShell that referenced this pull request Aug 6, 2026
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.

6 participants