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 github-pages dependency #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update github-pages dependency #58

wants to merge 1 commit into from

Conversation

RaevLogic
Copy link

Updated github-pages dependency to work with latest Ruby (2.5.1). This updated the Gemfile.lock, and I made changes to the _config.yml build settings.

Updated github-pages dependency to work with latest Ruby (2.5.1). This updated the Gemfile.lock, and I made changes to the _config.yml build settings.
@@ -49,9 +49,9 @@ other_nodegit_versions:
- 0.2.7

# Build settings
markdown: redcarpet
markdown: kramdown
Copy link
Member

Choose a reason for hiding this comment

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

Is there any specific reason why we needed to change the markdown or the highlighter settings?

Copy link
Author

@RaevLogic RaevLogic Sep 26, 2018

Choose a reason for hiding this comment

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

I was unable to get the website to generate without doing bundle update github-pages because of an error with the json dependency. Bundle updating fixed that error by updating everything in the Gemfile.lock file. The update got rid of redcarpet and pygments, so script/server wouldn't run because of missing dependencies. I googled what the alternatives were, and noticed rouge and kramdown were already in the updated Gemfile.lock. So I changed it in the config file, and script/server ran successfully, letting me generate the website.

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.

2 participants