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

Migrating docs from source to docs repo #6899

Merged
merged 3 commits into from
Jun 1, 2018

Conversation

sdwheeler
Copy link
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
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
Collaborator Author

@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
Collaborator

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

@sdwheeler
Copy link
Collaborator Author

@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
Collaborator

@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
Member

@iSazonov, @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
Collaborator

@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
Member

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

@sdwheeler
Copy link
Collaborator Author

@SteveL-MSFT @iSazonov that redirection only applies to GitHub Pages published using Jekyll.

@SteveL-MSFT
Copy link
Member

@sdwheeler too bad. For now, I would suggest moving ahead as-is. @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
Collaborator

iSazonov commented Jun 1, 2018

LGTM.

@adityapatwardhan adityapatwardhan merged commit 1f918bb into PowerShell:master Jun 1, 2018
@bergmeister
Copy link
Contributor

bergmeister commented Jun 2, 2018

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
Member

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

@sdwheeler
Copy link
Collaborator Author

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

@bergmeister
Copy link
Contributor

bergmeister commented Jun 4, 2018

@sdwheeler see pr #6981 and feel free to modify it to your liking

@rkeithhill
Copy link
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 sdwheeler deleted the sdw-docs2441 branch April 25, 2019 14:23
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.

None yet

6 participants