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

Configure Jekyll plugin jekyll-remote-theme to support Docker usage #999

Merged
merged 1 commit into from
Jul 9, 2019

Conversation

Lewiscowles1986
Copy link

@Lewiscowles1986 Lewiscowles1986 commented Jul 9, 2019

Description

I tried to build with Jekyll 3.6.x (jekyll/jekyll:latest). It went badly.

Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/main.scss':
                    File to import not found or unreadable: type-theme. Load paths: /faraday/docs/_sass /usr/local/bundle/gems/jekyll-theme-primer-0.5.2/_sass on line 2

By following the author guide https://github.com/benbalter/jekyll-remote-theme I noticed we were missing an entry.

After this things work for me.

Todos

  • Tests
  • Documentation

Additional Notes

I am using docker to build docs.

I tried to build with Jekyll 3.6.x (`jekyll/jekyll:latest`). It went badly.

By following the author guide https://github.com/benbalter/jekyll-remote-theme I noticed we were missing an entry.

After this things work for me.
Copy link
Member

@olleolleolle olleolleolle left a comment

Choose a reason for hiding this comment

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

This change allows more users to try out the website generation locally.

@Lewiscowles1986
Copy link
Author

Thank you @olleolleolle could you please check it does not break things for you locally?

It seems from the linked article from the author https://github.com/benbalter/jekyll-remote-theme that things will be going this way, but I like to know I'm not missing a trick.

@Lewiscowles1986 Lewiscowles1986 changed the title Follow author guide on remote-theme Add Jekyll plugin jekyll-remote-theme to support Docker usage Jul 9, 2019
@Lewiscowles1986 Lewiscowles1986 changed the title Add Jekyll plugin jekyll-remote-theme to support Docker usage Configure Jekyll plugin jekyll-remote-theme to support Docker usage Jul 9, 2019
@olleolleolle
Copy link
Member

olleolleolle commented Jul 9, 2019

@Lewiscowles1986 I tried it on my computer, by adding that line, and then running jekyll server. No change, still works.

$ jekyll --version
jekyll 3.8.5

@olleolleolle olleolleolle merged commit ac27ce5 into lostisland:master Jul 9, 2019
@olleolleolle
Copy link
Member

Thanks for contributing!

@Lewiscowles1986 Lewiscowles1986 deleted the patch-2 branch July 9, 2019 20:16
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