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

Conflicting quick-start instructions about assets #1268

Closed
asarkar opened this issue Sep 21, 2017 · 1 comment
Closed

Conflicting quick-start instructions about assets #1268

asarkar opened this issue Sep 21, 2017 · 1 comment

Comments

@asarkar
Copy link

asarkar commented Sep 21, 2017

I'm migrating a GitHub pages site to the Gem version. The Quick-Start instructions say:

Start by removing _includes, _layouts, _sass, assets folders

Later it says:

The preferred location is now /assets/images/

Obviously, if the location of images is supposed to be /assets/images/, the assets directory shouldn't be removed. But if the assets directory is not deleted, bundle exec jekyll serve fails with the following error:

      Generating... 
  Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/main.scss':
                    File to import not found or unreadable: variables. Load path: /var/lib/gems/2.3.0/gems/minimal-mistakes-jekyll-4.6.0/_sass on line 14
jekyll 3.5.2 | Error:  File to import not found or unreadable: variables.
Load path: /var/lib/gems/2.3.0/gems/minimal-mistakes-jekyll-4.6.0/_sass on line 14
@mmistakes
Copy link
Owner

You want to remove all of the files/folders within /assets, except anything you added as content. Meaning if you added images for posts and pages you'd leave those.

CSS, JavaScript, and fonts that are part of the forked theme need to be removed. Otherwise if you keep them and they exist in the gem version, they will be overrided.

Here's what's in the theme gem, if you have any of these files/folders in /assets, remove them.

https://github.com/mmistakes/minimal-mistakes/tree/master/assets

kkunapuli pushed a commit to kkunapuli/kkunapuli.github.io that referenced this issue May 30, 2019
Include asset folders that should be removed when migrating from a "forked theme".

Close mmistakes#1268
sumeetmondal pushed a commit to sumeetmondal/sumeetmondal.github.io that referenced this issue Sep 10, 2019
Include asset folders that should be removed when migrating from a "forked theme".

Close mmistakes#1268
makaroniame added a commit to makaroniame/makaroniame-old.github.io that referenced this issue May 18, 2022
Include asset folders that should be removed when migrating from a "forked theme".

Close mmistakes#1268
jchwenger pushed a commit to jchwenger/jchwenger.github.io that referenced this issue May 5, 2023
Include asset folders that should be removed when migrating from a "forked theme".

Close mmistakes#1268
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants