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

Update canonical URLs in v1.0.0/ docs on gh-pages #28821

Merged
merged 6 commits into from
Aug 22, 2018

Conversation

mortenpi
Copy link
Contributor

Changes the canonical URLs in v1.0.0/ and stable/ to https://docs.julialang.org/en/v1/ and creates a symlink v1/ -> stable/, making the "stable" docs accessible from that URL.

The URL update was done with a simple sed, so there should be no visible changes to the docs themselves.

This is the counterpart to #28792 to adhere to #26825.

find v1.0.0/ -name '*html' -exec sed -i 's/<link rel="canonical" href="https:\/\/docs.julialang.org\/en\/stable/<link rel="canonical" href="https:\/\/docs.julialang.org\/en\/v1/' {} \;
rm -R stable/ && cp -R v1.0.0/ stable/
ln -s stable v1
@mortenpi
Copy link
Contributor Author

Here is a readable diff of the v1.0.0/ directory generated with the help of tidy:
https://gist.github.com/mortenpi/bd60c2023516e0e19e8d2e565568f427#file-canonical-urls-diff

find v1.0.0/ v1.0.0-original/ -name '*.html' -exec tidy -m {} \;
diff -r -U1 v1.0.0-original/ v1.0.0/ > diff

This reverts commit 5e2ed12.

Let the master branch handle updating the version selector.
This reverts commit bd53b4e.

Actually, let's test the symlinks a bit before defaulting to them.
@fredrikekre fredrikekre merged commit 89e72ad into JuliaLang:gh-pages Aug 22, 2018
@mortenpi mortenpi deleted the gh-pages-update-to-v1 branch August 22, 2018 21:38
@fredrikekre
Copy link
Member

@mbauman mbauman added the docsystem The documentation building system label Aug 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docsystem The documentation building system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants