Skip to content
This repository has been archived by the owner on Jul 19, 2021. It is now read-only.

[Docs] Switch from Jekyll to github-pages gem #55

Merged
merged 1 commit into from
Nov 29, 2016
Merged

Conversation

cshold
Copy link
Contributor

@cshold cshold commented Nov 29, 2016

github-pages actually uses an older version of Jekyll, so some things break. This PR introduces the identical gem that the live site will use.

It also updates the docs/_config.yml file so markdown is rendered properly.
Lastly, page.url — on this version of Jekyll — includes index.html so the checks had to be updated for the sidebar active item.

This does not fix the SCSS import bug that kicked this search off.

@chrisberthe cc @m-ux

@cshold cshold self-assigned this Nov 29, 2016
@mcpatten
Copy link
Contributor

looks ok to me - I don't know enough about the gems / dependency tree though.

@cshold
Copy link
Contributor Author

cshold commented Nov 29, 2016

All from the switch of gem 'jekyll' to gem 'github-pages', group: :jekyll_plugins so should be smooth

@cshold cshold merged commit 78849e5 into master Nov 29, 2016
@cshold cshold deleted the gh-pages-gem branch November 29, 2016 19:00
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
t-kelly pushed a commit that referenced this pull request Jul 31, 2017
@lock
Copy link

lock bot commented Oct 26, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants