Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ashmaroli
Copy link
Member

Fix #215

Since users may have a footer.html at their source that still links to these removed includes and because Jekyll simply aborts if it can't find the requested include.

@ashmaroli
Copy link
Member Author

I recommend this be shipped and included in the github-pages gem a.s.a.p

@ashawley
Copy link
Contributor

The consequence of this change is that people with custom footer.html will not get the new social links. That's commendable.

This seems like a lot of flip-flopping. Hopefully, this doesn't risk making more confusion, rather than less. This avoids broken builds, but could also produce users who ask "Why can't I use the new social links?"

So will these includes be deleted in minima 3.0?

@DirtyF
Copy link
Member

DirtyF commented Mar 26, 2018

@ashawley The most important is not to break builds for the thousands of people that are using minima and don't care for the update. It's fair to say that when we ship 3.0, we will document the upgrade and remove those.

@DirtyF DirtyF merged commit b9fc2de into jekyll:master Mar 26, 2018
@ashawley
Copy link
Contributor

The most important is not to break builds for the thousands of people that are using minima

I'm just trying to discuss the change so we can review it.

I'm sure there are "thousands of people" using minima, but how many people have sourced the existing footer.html file?

@DirtyF DirtyF mentioned this pull request Mar 26, 2018
@DirtyF
Copy link
Member

DirtyF commented Mar 26, 2018

No matter how many people customized their footer, we don't want to break their site. We'll be more cautious next time and use post-install message to notify for breaking changes.

@ashawley
Copy link
Contributor

ashawley commented Mar 26, 2018

There appear to be about 30,000 public users of minima on Github alone:

https://github.com/search?q=minima+language%3AYAML&type=Code

But only a dozen over 68,000 people who have a footer.html referencing the old social files.

https://github.com/search?q=icon-twitter+path%3A_includes+language%3AHTML&type=Code

@mateuszbuda
Copy link

@ashawley You only count public repos.

@ashawley
Copy link
Contributor

ashawley commented Mar 26, 2018

You only count public repos.

Yes, I meant to qualify that and forgot. Thanks.

At that scale, you can understand the proportions/probability of a custom footer.html, regardless.

@ashmaroli ashmaroli deleted the reintroduce-social-includes branch March 27, 2018 07:17
@jekyllbot jekyllbot mentioned this pull request Mar 28, 2018
@nathanKramer
Copy link

FWIW this broke my build and was pretty frustrating trying to work out why without much help from jekyll's error message:

"Failure: A file was included in /_layouts/default.html that is a symlink or does not exist in your _includes directory."

I wasn't in your search @ashawley despite having a public repo so your number is probably low.

@ashmaroli
Copy link
Member Author

@nathanKramer This PR is a fix for the issue you mentioned.. Did you update to Minima-2.4.1? If you were simply referencing a past experience, then ignore this comment..

@ashawley
Copy link
Contributor

ashawley commented Apr 7, 2018

Is the github-pages gem going to get updated with this fix?

https://github.com/github/pages-gem/releases

@jekyll jekyll locked as spam and limited conversation to collaborators Apr 7, 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.

A file was included in /_layouts/default.html that is a symlink or does not exist in your _includes directory

6 participants